Skip to content

[wip] optimise consumption in order#15403

Closed
noob-se7en wants to merge 9 commits into
apache:masterfrom
noob-se7en:optimise_in_order
Closed

[wip] optimise consumption in order#15403
noob-se7en wants to merge 9 commits into
apache:masterfrom
noob-se7en:optimise_in_order

Conversation

@noob-se7en
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

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 28, 2025

Codecov Report

Attention: Patch coverage is 0% with 23 lines in your changes missing coverage. Please review.

Project coverage is 33.82%. Comparing base (59551e4) to head (edd8cde).
Report is 1946 commits behind head on master.

Files with missing lines Patch % Lines
...ore/data/manager/realtime/ConsumerCoordinator.java 0.00% 18 Missing ⚠️
...a/manager/realtime/RealtimeSegmentDataManager.java 0.00% 4 Missing ⚠️
...ata/manager/realtime/RealtimeTableDataManager.java 0.00% 1 Missing ⚠️

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

HEAD has 6 uploads less than BASE
Flag BASE (59551e4) HEAD (edd8cde)
skip-bytebuffers-false 7 6
unittests 5 3
unittests1 2 0
java-11 5 4
Additional details and impacted files
@@              Coverage Diff              @@
##             master   #15403       +/-   ##
=============================================
- Coverage     61.75%   33.82%   -27.93%     
- Complexity      207      690      +483     
=============================================
  Files          2436     2816      +380     
  Lines        133233   159601    +26368     
  Branches      20636    24484     +3848     
=============================================
- Hits          82274    53987    -28287     
- Misses        44911   101290    +56379     
+ Partials       6048     4324     -1724     
Flag Coverage Δ
custom-integration1 100.00% <ø> (+99.99%) ⬆️
integration 100.00% <ø> (+99.99%) ⬆️
integration1 100.00% <ø> (+99.99%) ⬆️
integration2 0.00% <ø> (ø)
java-11 33.80% <0.00%> (-27.91%) ⬇️
java-21 33.74% <0.00%> (-27.88%) ⬇️
skip-bytebuffers-false 33.82% <0.00%> (-27.93%) ⬇️
skip-bytebuffers-true 33.72% <0.00%> (+6.00%) ⬆️
temurin 33.82% <0.00%> (-27.93%) ⬇️
unittests 33.82% <0.00%> (-27.93%) ⬇️
unittests1 ?
unittests2 33.82% <0.00%> (+6.09%) ⬆️

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:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@noob-se7en noob-se7en closed this Apr 4, 2025
@xiangfu0 xiangfu0 added kubernetes Related to Kubernetes deployment ui UI related issue docker Related to Docker images or containerization labels Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Related to Docker images or containerization kubernetes Related to Kubernetes deployment ui UI related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants