release-25.2: sql/importer: fix TestUniqueUUID timeout#166344
release-25.2: sql/importer: fix TestUniqueUUID timeout#166344blathers-crl[bot] wants to merge 1 commit intorelease-25.2from
Conversation
Disable elastic CPU control in TestUniqueUUID to prevent the test from exceeding the 15m timeout. The test imports 50,000 rows across three tables with UUID/random primary key defaults. When the metamorphic constant `import-row-count-validation` selects `sync` mode, each import synchronously waits for an INSPECT validation job that scans the imported data. Combined with elastic CPU throttling (which was recently enabled for import jobs), the cumulative scan time across the three imports can exceed the test timeout. This mirrors the fix applied to other import tests in bda9189. Resolves: #166130 Epic: None Release note: None Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
55f322f to
8c358bf
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 #166344 is compliant with backport policy Confidence: high ❌ ENGREQ Check Failed: Please check one of the backport category checkboxes. 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
Backport 1/1 commits from #166309 on behalf of @mw5h.
Summary
Disable elastic CPU control in
TestUniqueUUIDto prevent the test fromexceeding the 15m timeout. The test imports 50,000 rows across three
tables with UUID/random primary key defaults.
When the metamorphic constant
import-row-count-validationselectssyncmode, each import synchronously waits for an INSPECT validationjob that scans the imported data. Combined with elastic CPU throttling
(which was recently enabled for import jobs), the cumulative scan time
across the three imports can exceed the test timeout.
This mirrors the fix applied to other import tests in bda9189.
Resolves: #166130
Epic: None
🤖 Generated with Claude Code
Release justification: