-
Notifications
You must be signed in to change notification settings - Fork 4.1k
release-25.2: workloadccl: mark tpccmultidb as bigInitialData #160452
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
release-25.2: workloadccl: mark tpccmultidb as bigInitialData #160452
Conversation
We already skip `tpcc` in `bigInitialData`, and I think it was a mistake that `tpccmultidb` wasn't marked in the same way. In particular, this will make it so that we don't run some of the test cases. The reason I was looking into this is that `TestAllRegisteredSetup/tpccmultidb` took 9 minutes on CI with external process mode. I couldn't quickly speed that up (e.g. exempting from rate limiting didn't help), but skipping it will work just fine. Release note: None
fc31299 to
2ce1030
Compare
|
Thanks for opening a backport. Before merging, please confirm that it falls into one of the following categories (select one):
Add a brief release justification to the PR description explaining your selection. Also, confirm that the change does not break backward compatibility and complies with all aspects of the backport policy. All backports must be reviewed by the TL and EM for the owning area. |
|
✅ PR #160452 is compliant with backport policy Confidence: high 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
Backport 1/1 commits from #160209 on behalf of @yuzefovich.
We already skip
tpccinbigInitialData, and I think it was a mistake thattpccmultidbwasn't marked in the same way. In particular, this will make it so that we don't run some of the test cases.The reason I was looking into this is that
TestAllRegisteredSetup/tpccmultidbtook 9 minutes on CI with external process mode. I couldn't quickly speed that up (e.g. exempting from rate limiting didn't help), but skipping it will work just fine.Epic: None
Release note: None
Release justification: test-only change.