release-25.4: sql/importer: fix TestUniqueUUID timeout#166345
release-25.4: sql/importer: fix TestUniqueUUID timeout#166345blathers-crl[bot] wants to merge 1 commit intorelease-25.4from
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>
1a0f4bf to
c38e602
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 #166345 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: