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

stress test trino-main #17490

Closed
wants to merge 5 commits into from
Closed

Conversation

findepi
Copy link
Member

@findepi findepi commented May 13, 2023

No description provided.

@cla-bot cla-bot bot added the cla-signed label May 13, 2023
@findepi findepi marked this pull request as draft May 13, 2023 13:11
@wendigo
Copy link
Contributor

wendigo commented May 13, 2023

@findepi nice!

@findepi
Copy link
Member Author

findepi commented May 13, 2023

test (core/trino-main, 20): #16933

io.trino.operator.TestRowNumberOperator.testMemoryReservationYield [bigint]; (took: 0.2 seconds)
2023-05-13T16:48:33.8445832Z java.lang.IllegalArgumentException: Invalid taskId another_query��.0.0

@findepi
Copy link
Member Author

findepi commented May 14, 2023

test (core/trino-main, 20, 43)
test (core/trino-main, 20, 50)
test (core/trino-main, 20, 56)
test (core/trino-main, 20, 68)

testMemoryReservationYield / testSemiJoinMemoryReservationYield in each of them.

@findepi
Copy link
Member Author

findepi commented May 14, 2023

253 successful and 5 failing checks
ci / test (core/trino-main, 20, 26) (pull_request) Failing after 28m
ci / test (core/trino-main, 20, 36) (pull_request) Failing after 28m
ci / test (core/trino-main, 20, 63) (pull_request) Failing after 28m
ci / test (core/trino-main, 20, 76) (pull_request) Failing after 28m
ci / test (core/trino-main, 20, 84) (pull_request) Failing after 28m

https://github.com/trinodb/trino/actions/runs/4971217839/jobs/8895586394?pr=17490

@findepi findepi closed this May 15, 2023
@findepi findepi deleted the findepi/main-stress branch May 15, 2023 08:07
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.

2 participants