Skip to content

PHOENIX-7836 UncoveredGlobalIndexRegionScanner2IT relax per-test query timeout - #2453

Open
apurtell wants to merge 1 commit into
apache:masterfrom
apurtell:PHOENIX-7836
Open

PHOENIX-7836 UncoveredGlobalIndexRegionScanner2IT relax per-test query timeout#2453
apurtell wants to merge 1 commit into
apache:masterfrom
apurtell:PHOENIX-7836

Conversation

@apurtell

Copy link
Copy Markdown
Contributor

testCount upserts 100,000 rows, builds an uncovered global index, sets phoenix.scanner.disable.uncovered.regions.move=false, and runs an aggregate count that forces every region scan through the post-dummy region-move callback. For the uncovered=false, salted=true parameterization the per-region work is heavier and the global phoenix.query.timeoutMs set by QueryServicesTestImpl to DEFAULT_THREAD_TIMEOUT_MS (5 minutes) can be exhausted. The fix overrides the timeout only for this test's connection by passing a Properties with phoenix.query.timeoutMs = 20 * 60 * 1000.

@palashc

Copy link
Copy Markdown
Contributor

@palashcpalashc 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.

+1, pending build results

@apurtell
apurtell requested a balanced review from CopilotAugust 15, 2026 12:17

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@apurtell@palashc