Skip to content

Add timeout overflow break handling at leaf stage - #17216

Open
xiangfu0 wants to merge 1 commit into
apache:masterfrom
xiangfu0:feature/timeout-overflow-leaf
Open

Add timeout overflow break handling at leaf stage#17216
xiangfu0 wants to merge 1 commit into
apache:masterfrom
xiangfu0:feature/timeout-overflow-leaf

Conversation

@xiangfu0

@xiangfu0 xiangfu0 commented Nov 15, 2025

Copy link
Copy Markdown
Contributor

Summary

  • introduce timeoutOverflowMode plumbing in server/broker metadata
  • make LeafOperator break early only if mode=BREAK and expose a TIMEOUT_OVERFLOW_REACHED stat so intermediates keep running
  • add SlowMockInstanceDataManagerFactory plus TimeoutOverflowQueryRunnerTest to cover simple select, join, and aggregation cases

query with partial results
image
query with full results
image

@codecov-commenter

codecov-commenter commented Nov 15, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 58.59873% with 65 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.59%. Comparing base (c500e4f) to head (e8aa877).

Files with missing lines Patch % Lines
...pinot/query/runtime/operator/BaseJoinOperator.java 25.00% 18 Missing and 3 partials ⚠️
...requesthandler/MultiStageBrokerRequestHandler.java 0.00% 16 Missing ⚠️
...che/pinot/query/runtime/operator/LeafOperator.java 71.42% 7 Missing and 5 partials ⚠️
...va/org/apache/pinot/query/runtime/QueryRunner.java 80.43% 4 Missing and 5 partials ⚠️
...common/response/broker/BrokerResponseNativeV2.java 70.00% 1 Missing and 2 partials ⚠️
...not/query/runtime/operator/MultiStageOperator.java 71.42% 1 Missing and 1 partial ⚠️
...va/org/apache/pinot/spi/utils/CommonConstants.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #17216      +/-   ##
============================================
- Coverage     65.60%   65.59%   -0.02%     
+ Complexity     1423     1417       -6     
============================================
  Files          3450     3450              
  Lines        218378   218521     +143     
  Branches      34670    34700      +30     
============================================
+ Hits         143275   143343      +68     
- Misses        63538    63604      +66     
- Partials      11565    11574       +9     
Flag Coverage Δ
custom-integration1 ?
integration ?
integration1 ?
integration2 ?
java-25 65.59% <58.59%> (-0.02%) ⬇️
temurin 65.59% <58.59%> (-0.02%) ⬇️
unittests 65.59% <58.59%> (-0.01%) ⬇️
unittests1 57.00% <65.24%> (+<0.01%) ⬆️
unittests2 37.88% <4.45%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xiangfu0
xiangfu0 force-pushed the feature/timeout-overflow-leaf branch 3 times, most recently from 53029b6 to d04720b Compare November 17, 2025 08:35
@xiangfu0
xiangfu0 marked this pull request as ready for review November 17, 2025 09:31
@xiangfu0
xiangfu0 requested a review from Copilot November 17, 2025 09:36

Copilot AI 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@xiangfu0
xiangfu0 force-pushed the feature/timeout-overflow-leaf branch 2 times, most recently from 04d6211 to ccaacce Compare November 17, 2025 13:04
@xiangfu0
xiangfu0 force-pushed the feature/timeout-overflow-leaf branch 8 times, most recently from e2a0438 to c346cfb Compare July 25, 2026 08:02
@xiangfu0
xiangfu0 force-pushed the feature/timeout-overflow-leaf branch 4 times, most recently from 892a535 to 97328f2 Compare July 29, 2026 08:02
@xiangfu0
xiangfu0 force-pushed the feature/timeout-overflow-leaf branch from 97328f2 to e8aa877 Compare July 30, 2026 08:03
Sign up for free to 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