Skip to content

[ci] Decrease the concurrent workers for Yugabyte' with -enable_scd_hash_lock flag test - #1659

Open
MariemBaccari wants to merge 17 commits into
interuss:masterfrom
Orbitalize:decrease_cc_workers
Open

MariemBaccari wants to merge 17 commits into
interuss:masterfrom
Orbitalize:decrease_cc_workers

Conversation

@MariemBaccari

@MariemBaccari MariemBaccari commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Following the extension of performance testing in #1618, the CI runs against Yugabyte with the -enable_scd_hash_lock flag can face intermittent load-sensitive test failures due to the high number of concurrent workers.
This PR fixes this by decreasing the --heavy_traffic_concurrent_workers flag to 1.

@MariemBaccari MariemBaccari changed the title [testing] decrease --heavy_traffic_concurrent_workers for yugabyte [ci] decrease --heavy_traffic_concurrent_workers for yugabyte Aug 21, 2026
@MariemBaccari
MariemBaccari marked this pull request as draft August 21, 2026 09:33
@MariemBaccari MariemBaccari changed the title [ci] decrease --heavy_traffic_concurrent_workers for yugabyte [ci] Decrease the concurrent workers for Yugabyte' with -enable_scd_hash_lock flag test Aug 21, 2026
@MariemBaccari
MariemBaccari marked this pull request as ready for review August 21, 2026 10:06
@MariemBaccari
MariemBaccari marked this pull request as draft August 21, 2026 11:37
@MariemBaccari
MariemBaccari marked this pull request as ready for review September 2, 2026 10:22

@mickmis mickmis left a comment

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.

This PR fixes this by decreasing the --heavy_traffic_concurrent_workers flag to 5.

This PR implements 1 concurrent worker. Which is it?

Also, I've seen that happen with CRDB too (although much less frequently). Have you observed the same thing?

Otherwise LGTM.

@MariemBaccari

Copy link
Copy Markdown
Contributor Author

@mickmis It decreases to 1, I just updated the PR description sorry. I haven't had an issues with CRDB so far.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants