Skip to content
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

Implement retryable clickhouse queries #4355

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

IvanIvanoff
Copy link
Member

@IvanIvanoff IvanIvanoff commented Jul 26, 2024

Changes

In case there is no problem with syntax, table/column names or other non-retryable errors, retry Clickhouse queries once, before returning an error.

This will also retry some timeout errors. In case of API calls and timeout errors, the process running the API query and the database query will be terminated, so the retried query will also be terminated.

Ticket

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have tried to find clearer solution before commenting hard-to-understand parts of code
  • I have added tests that prove my fix is effective or that my feature works

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants