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

Run JDBC connector tests with H2 in single thread #7210

Conversation

findepi
Copy link
Member

@findepi findepi commented Mar 8, 2021

H2 DDL operations can sometimes take a global lock, leading to apparent
deadlocks.

Fixes #7209

@cla-bot cla-bot bot added the cla-signed label Mar 8, 2021
@findepi findepi requested review from losipiuk and kokosing March 8, 2021 10:08
@losipiuk
Copy link
Member

losipiuk commented Mar 8, 2021

Do you happen to know how much does it add to test runtime?

@findepi
Copy link
Member Author

findepi commented Mar 8, 2021

Do you happen to know how much does it add to test runtime?

i do not

@kokosing
Copy link
Member

kokosing commented Mar 8, 2021

Do you happen to know how much does it add to test runtime?

H2 is quite fast. We run tests in 2 threads IIRC and we have to tests classes so there is an overlap still. It should not affect much.

H2 DDL operations can sometimes take a global lock, leading to apparent
deadlocks.
@findepi findepi force-pushed the findepi/run-jdbc-connector-tests-with-h2-in-single-thread-70358c branch from ecec7d0 to d4fdb4e Compare March 8, 2021 13:06
@findepi findepi merged commit 300dc28 into trinodb:master Mar 8, 2021
@findepi findepi deleted the findepi/run-jdbc-connector-tests-with-h2-in-single-thread-70358c branch March 8, 2021 21:52
@findepi findepi added this to the 354 milestone Mar 8, 2021
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.

Flaky test TestJdbcConnectorTest.testDropColumn
3 participants