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

Retry product tests on MapRedTask ClosedByInterruptException failures #15879

Merged
merged 3 commits into from
Jan 30, 2023

Conversation

findepi
Copy link
Member

@findepi findepi commented Jan 27, 2023

Retries for #5427 (close #5427)

With stacktrace in the middle, it's hard to find the end of the message,
so it's hard to read it as a sentence.
@findepi findepi added no-release-notes This pull request does not require release notes entry dx Issues or pull requests related to developer experience labels Jan 27, 2023
@cla-bot cla-bot bot added the cla-signed label Jan 27, 2023
@findepi findepi self-assigned this Jan 27, 2023
/**
* Link to issues:
* <ul>
* <ol><a href="https://github.com/trinodb/trino/issues/4936">#4936</a></ol>
Copy link
Member Author

Choose a reason for hiding this comment

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

* Link to issues:
* <ul>
* <ol><a href="https://github.com/trinodb/trino/issues/4936">#4936</a></ol>
* <ol><a href="https://github.com/trinodb/trino/issues/5427">#5427</a></ol>
Copy link
Member Author

Choose a reason for hiding this comment

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

Writing through Hive can fail in ways other than just "could only be
replicated to 0 nodes instead of minReplication". Let's make the names
of constants more generic to support more cases in the future, and also
abstract what the underlying problem may be.
@findepi findepi force-pushed the findepi/channel-closed branch from d4f25b4 to c9ec513 Compare January 27, 2023 13:59
@findepi findepi mentioned this pull request Jan 27, 2023
@findepi
Copy link
Member Author

findepi commented Jan 30, 2023

CI #12535

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed dx Issues or pull requests related to developer experience no-release-notes This pull request does not require release notes entry
Development

Successfully merging this pull request may close these issues.

Flaky: product tests fail with MapRedTask. Error caching map.xml: java.nio.channels.ClosedByInterruptException
2 participants