Skip to content

Use Failsafe to retry instead iceberg's Tasks in iceberg connector#15758

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
krvikash:use-FailSafe-in-iceberg
Jan 19, 2023
Merged

Use Failsafe to retry instead iceberg's Tasks in iceberg connector#15758
ebyhr merged 1 commit intotrinodb:masterfrom
krvikash:use-FailSafe-in-iceberg

Conversation

@krvikash
Copy link
Contributor

Description

Inspired from #14869 (comment)

Release notes

(X) This is not user-visible or docs only and no release notes are required.

@cla-bot cla-bot bot added the cla-signed label Jan 18, 2023
@krvikash krvikash added the no-release-notes This pull request does not require release notes entry label Jan 18, 2023
Copy link
Member

@alexjo2144 alexjo2144 left a comment

Choose a reason for hiding this comment

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

👍

@krvikash krvikash force-pushed the use-FailSafe-in-iceberg branch from d9096c5 to 7f8d928 Compare January 18, 2023 18:24
@krvikash
Copy link
Contributor Author

Rebased with upstream master.

@krvikash krvikash force-pushed the use-FailSafe-in-iceberg branch from 7f8d928 to d5e93aa Compare January 19, 2023 03:16
@krvikash
Copy link
Contributor Author

Addressed comments.

@ebyhr ebyhr merged commit 7facf0e into trinodb:master Jan 19, 2023
@krvikash krvikash deleted the use-FailSafe-in-iceberg branch January 19, 2023 07:00
@github-actions github-actions bot added this to the 406 milestone Jan 19, 2023
@findepi
Copy link
Member

findepi commented Jan 19, 2023

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed no-release-notes This pull request does not require release notes entry

Development

Successfully merging this pull request may close these issues.

4 participants