Skip to content

Release subscription WAL buffers between prefetch rounds (#18956) - #19255

Merged
JackieTien97 merged 1 commit into
rc/2.0.12from
sub-sub
Nov 13, 2026
Merged

JackieTien97 merged 1 commit into
rc/2.0.12from
sub-sub

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Description

As the title said.


This PR has:

  • been self-reviewed.
    • concurrent read
    • concurrent write
    • concurrent read and write
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods.
  • added or updated version, license, or notice information
  • added comments explaining the "why" and the intent of the code wherever would not be obvious
    for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold
    for code coverage.
  • added integration tests.
  • been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

* Release subscription WAL read buffers between prefetch rounds

* Stream Prometheus snapshots in bounded UTF-8 chunks

* Handle shortened WAL segments after buffer release
@JackieTien97
JackieTien97 merged commit 5b9119d into rc/2.0.12 Nov 13, 2026
37 of 39 checks passed
@JackieTien97
JackieTien97 deleted the sub-sub branch November 13, 2026 04:29
@codecov

codecov Bot commented Nov 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.01887% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 44.02%. Comparing base (78d4c7f) to head (ab6777d).
⚠️ Report is 3 commits behind head on rc/2.0.12.

Files with missing lines Patch % Lines
...on/broker/consensus/ConsensusPrefetchingQueue.java 0.00% 6 Missing ⚠️
...cription/broker/consensus/ProgressWALIterator.java 0.00% 3 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##             rc/2.0.12   #19255      +/-   ##
===============================================
+ Coverage        43.97%   44.02%   +0.05%     
- Complexity         458      505      +47     
===============================================
  Files             5468     5468              
  Lines           398624   398675      +51     
  Branches         52454    52464      +10     
===============================================
+ Hits            175298   175526     +228     
+ Misses          223326   223149     -177     

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

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