Skip to content

Disable flaky testDeleteRowsConcurrently in Iceberg JDBC catalog#22933

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
ebyhr:ebi/iceberg-disable-jdbc-delete
Aug 7, 2024
Merged

Disable flaky testDeleteRowsConcurrently in Iceberg JDBC catalog#22933
ebyhr merged 1 commit intotrinodb:masterfrom
ebyhr:ebi/iceberg-disable-jdbc-delete

Conversation

@ebyhr
Copy link
Copy Markdown
Member

@ebyhr ebyhr commented Aug 5, 2024

Description

Relates to #21862

Release notes

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

@cla-bot cla-bot bot added the cla-signed label Aug 5, 2024
@github-actions github-actions bot added the iceberg Iceberg connector label Aug 5, 2024
@ebyhr ebyhr requested review from hashhar and wendigo August 5, 2024 13:46
@Override
public void testDeleteRowsConcurrently()
{
// TODO https://github.com/trinodb/trino/issues/21862 Fix flaky test
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we know why this is failing?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Nope. I guess the engine was busy because of concurrent tests.

@ebyhr ebyhr merged commit 130af7a into trinodb:master Aug 7, 2024
@ebyhr ebyhr deleted the ebi/iceberg-disable-jdbc-delete branch August 7, 2024 04:33
@github-actions github-actions bot added this to the 454 milestone Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed iceberg Iceberg connector

Development

Successfully merging this pull request may close these issues.

2 participants