Skip to content

tidb: add singleton_sketch for RowSampleCollector#407

Open
0xPoe wants to merge 1 commit intopingcap:masterfrom
0xPoe:poe-patch-f1-2026-03-31
Open

tidb: add singleton_sketch for RowSampleCollector#407
0xPoe wants to merge 1 commit intopingcap:masterfrom
0xPoe:poe-patch-f1-2026-03-31

Conversation

@0xPoe
Copy link
Copy Markdown
Member

@0xPoe 0xPoe commented Mar 31, 2026

What problem does this PR solve?

Issue Number: ref pingcap/tidb#67449

Problem Summary:

What is changed and how it works?

What's Changed:

Added the singleton sketch to help us collect the singleton values from each TiKV.

How it Works:

Some preparatory work.

Related changes

  • PR to update pingcap/docs/pingcap/docs-cn:
  • Need to cherry-pick to the release branch

Copy link
Copy Markdown
Member Author

@0xPoe 0xPoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔢 Self-check (PR reviewed by myself and ready for feedback)

  • Code compiles successfully

  • Unit tests added

  • No AI-generated elegant nonsense in PR.

  • Bazel files updated

  • Comments added where necessary

  • PR title and description updated

  • Documentation PR created (or confirmed not needed)

  • PR size is reasonable

/cc @henrybw @winoros

@0xPoe 0xPoe changed the title feat: f1 sketch tidb: add f1_sketch for RowSampleCollector Mar 31, 2026
@0xPoe 0xPoe force-pushed the poe-patch-f1-2026-03-31 branch 2 times, most recently from a7025da to c976021 Compare March 31, 2026 07:40
@0xPoe 0xPoe requested review from henrybw and winoros March 31, 2026 07:40
winoros
winoros previously approved these changes Mar 31, 2026
@0xPoe
Copy link
Copy Markdown
Member Author

0xPoe commented Mar 31, 2026

/hold

@0xPoe 0xPoe changed the title tidb: add f1_sketch for RowSampleCollector DNM: tidb: add f1_sketch for RowSampleCollector Mar 31, 2026
Copy link
Copy Markdown

@henrybw henrybw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me.

@0xPoe 0xPoe changed the title DNM: tidb: add f1_sketch for RowSampleCollector tidb: add f1_sketch for RowSampleCollector Apr 1, 2026
@0xPoe
Copy link
Copy Markdown
Member Author

0xPoe commented Apr 1, 2026

/unhold

Signed-off-by: 0xPoe <techregister@pm.me>
@0xPoe 0xPoe force-pushed the poe-patch-f1-2026-03-31 branch from 10f71a5 to 3def234 Compare April 2, 2026 07:56
@0xPoe 0xPoe changed the title tidb: add f1_sketch for RowSampleCollector tidb: add singleton_sketch for RowSampleCollector Apr 2, 2026
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.

3 participants