Skip to content

Comments

Add max error retry config option to glue client#4611

Merged
sopel39 merged 1 commit intotrinodb:masterfrom
vincentpoon:maxErrorRetries
Jul 30, 2020
Merged

Add max error retry config option to glue client#4611
sopel39 merged 1 commit intotrinodb:masterfrom
vincentpoon:maxErrorRetries

Conversation

@vincentpoon
Copy link
Member

No description provided.

@cla-bot cla-bot bot added the cla-signed label Jul 28, 2020
Copy link
Member

Choose a reason for hiding this comment

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

is 10 default glue value?

Copy link
Member Author

Choose a reason for hiding this comment

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

Default is no retries. I used 10 because that's the same default Presto sets for hive.s3.max-error-retries (which sets the same property on AWS's ClientConfiguration)

@vincentpoon
Copy link
Member Author

Updated the hive.rst documentation as well

@sopel39 sopel39 merged commit 54d175f into trinodb:master Jul 30, 2020
@sopel39
Copy link
Member

sopel39 commented Jul 30, 2020

merged, thanks!

@sopel39 sopel39 mentioned this pull request Jul 30, 2020
8 tasks
aweisberg pushed a commit to v-jizhang/presto that referenced this pull request May 12, 2021
Cherry pick of trinodb/trino#4611 and
https://github.com/vincentpoon/prestosql/commit/c1ac9ac257bc5a07f32a359c7aac6735fd6ef69f#diff-824c0eed6933e963a65ba98d9bc3582e5f3f96fbd78982996327b10e54bd5c67

Co-authored-by: Vincent Poon <vincent.poon@salesforce.com>
Co-authored-by: Philippe Gagnon <pgagnon@users.noreply.github.com>
arhimondr pushed a commit to prestodb/presto that referenced this pull request May 13, 2021
Cherry pick of trinodb/trino#4611 and
https://github.com/vincentpoon/prestosql/commit/c1ac9ac257bc5a07f32a359c7aac6735fd6ef69f#diff-824c0eed6933e963a65ba98d9bc3582e5f3f96fbd78982996327b10e54bd5c67

Co-authored-by: Vincent Poon <vincent.poon@salesforce.com>
Co-authored-by: Philippe Gagnon <pgagnon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants