-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Context aware error types based on ClickHouse error codes #250
Comments
Without such context-aware error types, one garbage span can block the whole trace ingestion. |
Not every error is retryable. Some errors are permanent and retrying them makes no sense. The sync mode today retries all the errors forever and blocks everything else from getting ingested. There should be an error type-based retry mechanism and the exporters should return context aware errors to the caller instead of just returning err as is from the clickhouse-go client.
The text was updated successfully, but these errors were encountered: