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

Apply INCREASE_CONTAINER_SIZE execution strategy for user supplied error codes #21513

Merged

Conversation

pgupta2
Copy link
Contributor

@pgupta2 pgupta2 commented Dec 11, 2023

INCREASE_CONTAINER_SIZE execution strategy is quite generic
and can be applied to multiple error codes. Adding a session
property to pass a string of comma separated error_codes which
should be retried with this execution strategy.

@pgupta2 pgupta2 requested review from shrinidhijoshi and a team as code owners December 11, 2023 21:00
@pgupta2 pgupta2 force-pushed the retry_increased_container_error_codes branch from 6b6e680 to ad382a7 Compare December 11, 2023 21:01
@pgupta2 pgupta2 force-pushed the retry_increased_container_error_codes branch 2 times, most recently from 212acf7 to 03b6f71 Compare December 11, 2023 21:45
Copy link
Contributor

@singcha singcha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating the PR.

@pgupta2 pgupta2 force-pushed the retry_increased_container_error_codes branch from 03b6f71 to dda3e56 Compare December 12, 2023 00:18
@pgupta2 pgupta2 force-pushed the retry_increased_container_error_codes branch from dda3e56 to 78dd8c3 Compare December 12, 2023 02:42
@pgupta2 pgupta2 force-pushed the retry_increased_container_error_codes branch from 78dd8c3 to c166b05 Compare December 12, 2023 03:23
@pgupta2 pgupta2 requested review from shrinidhijoshi and removed request for arhimondr and presto-oss December 12, 2023 04:45
@pgupta2 pgupta2 merged commit e650152 into prestodb:master Dec 12, 2023
56 checks passed
@wanglinsong wanglinsong mentioned this pull request Feb 12, 2024
64 tasks
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.

4 participants