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

Fix trino-tests timeout #13618

Merged
merged 4 commits into from
Aug 12, 2022
Merged

Conversation

findepi
Copy link
Member

@findepi findepi commented Aug 11, 2022

No description provided.

@findepi findepi force-pushed the findepi/fix-trino-tests-timeout branch from d917476 to 288239a Compare August 11, 2022 14:01
@findepi
Copy link
Member Author

findepi commented Aug 11, 2022

I moved TestFaultTolerantExecutionDynamicFiltering too

`trino-faulttolerant-tests` is the module for fault-tolerant tests, so
move `TestDistributedFaultTolerantEngineOnlyQueries` and
`TestFaultTolerantExecutionDynamicFiltering` from `trino-tests` there.

This also isolates these test classes when executing on CI. It was seen
that `trino-tests` often times out on the CI, apparently because of
`TestDistributedFaultTolerantEngineOnlyQueries`, leading to
`trino-tests` tests results being not available.
@findepi findepi force-pushed the findepi/fix-trino-tests-timeout branch from 288239a to 6fb478f Compare August 11, 2022 15:37
@arhimondr
Copy link
Contributor

Thanks @findepi

@findepi
Copy link
Member Author

findepi commented Aug 12, 2022

CI #13638, TestDeltaLakeSharedHiveMetastoreWithViews.init timeout, ..

@findepi findepi merged commit 766da86 into trinodb:master Aug 12, 2022
@findepi findepi deleted the findepi/fix-trino-tests-timeout branch August 12, 2022 10:10
@github-actions github-actions bot added this to the 393 milestone Aug 12, 2022
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.

6 participants