Skip to content

mv2 draft#18019

Draft
hongkunxu wants to merge 1 commit intoapache:masterfrom
hongkunxu:feat/mv2
Draft

mv2 draft#18019
hongkunxu wants to merge 1 commit intoapache:masterfrom
hongkunxu:feat/mv2

Conversation

@hongkunxu
Copy link
Copy Markdown
Contributor

Instructions:

  1. The PR has to be tagged with at least one of the following labels (*):
    1. feature
    2. bugfix
    3. performance
    4. ui
    5. backward-incompat
    6. release-notes (**)
  2. Remove these instructions before publishing the PR.

(*) Other labels to consider:

  • testing
  • dependencies
  • docker
  • kubernetes
  • observability
  • security
  • code-style
  • extension-point
  • refactor
  • cleanup

(**) Use release-notes label for scenarios like:

  • New configuration options
  • Deprecation of configurations
  • Signature changes to public methods/interfaces
  • New plugins added or old plugins removed

Signed-off-by: Hongkun Xu <xuhongkun666@163.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 29, 2026

Codecov Report

❌ Patch coverage is 71.25000% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.56%. Comparing base (a3c27e2) to head (dafc828).
⚠️ Report is 123 commits behind head on master.

Files with missing lines Patch % Lines
...t/common/minion/MaterializedViewMetadataUtils.java 0.00% 16 Missing ⚠️
.../pinot/common/minion/MaterializedViewMetadata.java 90.19% 4 Missing and 1 partial ⚠️
...ache/pinot/common/metadata/ZKMetadataProvider.java 0.00% 1 Missing ⚠️
.../org/apache/pinot/core/common/MinionConstants.java 0.00% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (a3c27e2) and HEAD (dafc828). Click for more details.

HEAD has 32 uploads less than BASE
Flag BASE (a3c27e2) HEAD (dafc828)
unittests2 2 0
unittests 4 2
java-11 5 1
temurin 10 2
integration 6 0
java-21 5 1
integration1 2 0
integration2 2 0
custom-integration1 2 0
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #18019      +/-   ##
============================================
- Coverage     63.23%   55.56%   -7.68%     
+ Complexity     1481      746     -735     
============================================
  Files          3190     2505     -685     
  Lines        192285   143192   -49093     
  Branches      29470    22976    -6494     
============================================
- Hits         121598    79563   -42035     
+ Misses        61156    56895    -4261     
+ Partials       9531     6734    -2797     
Flag Coverage Δ
custom-integration1 ?
integration ?
integration1 ?
integration2 ?
java-11 55.50% <71.25%> (-7.71%) ⬇️
java-21 55.54% <71.25%> (-7.67%) ⬇️
temurin 55.56% <71.25%> (-7.68%) ⬇️
unittests 55.56% <71.25%> (-7.68%) ⬇️
unittests1 55.56% <71.25%> (+<0.01%) ⬆️
unittests2 ?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants