Skip to content

CXXCBC-543: Add retries for columnar query#637

Merged
avsej merged 2 commits intocouchbase:mainfrom
DemetrisChr:columnar-retry
Aug 12, 2024
Merged

CXXCBC-543: Add retries for columnar query#637
avsej merged 2 commits intocouchbase:mainfrom
DemetrisChr:columnar-retry

Conversation

@DemetrisChr
Copy link
Copy Markdown
Contributor

@DemetrisChr DemetrisChr commented Jul 25, 2024

  • Refactor columnar query execution into a pending_query_operation in query_component, to enable handling retries
  • Implement exponential backoff with full jitter and use it for columnar query retries if all errors have is_retriable == true
  • Add a randomly generated client-context-id to the query request (include in payload & logging)
  • Extend session_manager->check_out() to allow specifying an 'undesired' node. If other nodes for the required service exist, any other node will be selected at random. This is used for columnar query retries.

@DemetrisChr DemetrisChr changed the title Add retries for columnar query CXXCBC-543: Add retries for columnar query Jul 25, 2024
@DemetrisChr DemetrisChr force-pushed the columnar-retry branch 3 times, most recently from dc41491 to 4aa0b52 Compare August 5, 2024 15:38
@DemetrisChr DemetrisChr marked this pull request as ready for review August 5, 2024 16:11
@DemetrisChr DemetrisChr force-pushed the columnar-retry branch 2 times, most recently from b88e482 to 781a475 Compare August 9, 2024 16:53
@avsej avsej merged commit 51500f1 into couchbase:main Aug 12, 2024
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