Skip to content

branch-4.2: [fix](be) Stop preallocating compression-sized t-digest b… - #69470

Merged
yiguolei merged 1 commit into
apache:branch-4.2from
mrhhsg:pick_69284
Oct 23, 2026
Merged

yiguolei merged 1 commit into
apache:branch-4.2from
mrhhsg:pick_69284

Conversation

@mrhhsg

@mrhhsg mrhhsg commented Oct 22, 2026

Copy link
Copy Markdown
Member

…uffers per group #69284

Cherry-picked from #69284 (c9bd8b0).

Conflict resolution: preserved branch-4.2 TDigest camelCase method names and adapted the new threshold assertions to maxProcessed()/maxUnprocessed(). Kept the branch-4.2 quantile-state test structure while replacing its eager-capacity assertion. No unrelated master changes were imported.

(cherry picked from commit c9bd8b0)

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

Release note

None

Check List (For Author)

  • Test

    • Regression test
    • Unit Test
    • Manual test (add detailed scripts or steps below)
    • No need to test or manual test. Explain why:
      • This is a refactor/code format and no logic has been changed.
      • Previous test can cover this change.
      • No code files have been changed.
      • Other reason
  • Behavior changed:

    • No.
    • Yes.
  • Does this need documentation?

    • No.
    • Yes.

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

…uffers per group apache#69284

Cherry-picked from apache#69284 (c9bd8b0).

Conflict resolution: preserved branch-4.2 TDigest camelCase method names and adapted the new threshold assertions to maxProcessed()/maxUnprocessed(). Kept the branch-4.2 quantile-state test structure while replacing its eager-capacity assertion. No unrelated master changes were imported.

(cherry picked from commit c9bd8b0)
@mrhhsg
mrhhsg requested a review from yiguolei as a code owner October 22, 2026 12:28
@mrhhsg

mrhhsg commented Oct 22, 2026

Copy link
Copy Markdown
Member Author

run buildall

@hello-stephen

Copy link
Copy Markdown
Contributor

BE Regression && UT Coverage Report

Increment line coverage 100% (0/0) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 76.36% (34494/45175)
Line Coverage 61.21% (389729/636675)
Region Coverage 58.04% (328403/565848)
Branch Coverage 58.96% (151496/256937)

@yiguolei
yiguolei merged commit c90d0c8 into apache:branch-4.2 Oct 23, 2026
29 of 30 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@github-actions github-actions Bot added the approved Indicates a PR has been approved by one committer. label Oct 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

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

Labels

approved Indicates a PR has been approved by one committer. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants