Skip to content

Do not pass server metrics in PlanMaker#17808

Open
Jackie-Jiang wants to merge 1 commit intoapache:masterfrom
Jackie-Jiang:query_plan
Open

Do not pass server metrics in PlanMaker#17808
Jackie-Jiang wants to merge 1 commit intoapache:masterfrom
Jackie-Jiang:query_plan

Conversation

@Jackie-Jiang
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Mar 4, 2026

Codecov Report

❌ Patch coverage is 42.85714% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.26%. Comparing base (5df2ffb) to head (819390e).

Files with missing lines Patch % Lines
...va/org/apache/pinot/core/plan/maker/PlanMaker.java 0.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #17808      +/-   ##
============================================
+ Coverage     63.23%   63.26%   +0.03%     
  Complexity     1456     1456              
============================================
  Files          3186     3187       +1     
  Lines        191613   191618       +5     
  Branches      29314    29314              
============================================
+ Hits         121165   121231      +66     
+ Misses        60967    60909      -58     
+ Partials       9481     9478       -3     
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (?)
java-11 63.24% <42.85%> (+0.03%) ⬆️
java-21 63.21% <42.85%> (+0.01%) ⬆️
temurin 63.26% <42.85%> (+0.03%) ⬆️
unittests 63.26% <42.85%> (+0.03%) ⬆️
unittests1 55.62% <42.85%> (+0.02%) ⬆️
unittests2 34.16% <0.00%> (+0.01%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 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.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants