Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Unify the snapshot read and latest read-1 #21441

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Unify the snapshot read and latest read-1 #21441

wants to merge 15 commits into from
+587 −730

Conversation

triump2020
Copy link
Contributor

@triump2020 triump2020 commented Feb 24, 2025 •
edited
Loading

What type of PR is this?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • [ x] Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #19347

What this PR does / why we need it:

  1. Unify the snasphot read and latest read
  2. Modified the implementation of stats-related code.

@matrix-meow matrix-meow added the size/M Denotes a PR that changes [100,499] lines label Feb 24, 2025
@triump2020 triump2020 force-pushed the preprare-for-snapshot-read-refactor-1 branch from aff4c66 to 70fbc15 Compare February 24, 2025 16:29
@matrix-meow matrix-meow added size/XL Denotes a PR that changes [1000, 1999] lines and removed size/L Denotes a PR that changes [500,999] lines labels Mar 6, 2025
@triump2020 triump2020 changed the title Prepare for snapshot read refactor 1 Unify the snapshot read and latest read-1 Mar 6, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@XuPeng-SH XuPeng-SH Awaiting requested review from XuPeng-SH XuPeng-SH is a code owner

@volgariver6 volgariver6 Awaiting requested review from volgariver6 volgariver6 is a code owner

@zhangxu19830126 zhangxu19830126 Awaiting requested review from zhangxu19830126 zhangxu19830126 is a code owner

@reusee reusee Awaiting requested review from reusee reusee is a code owner

@daviszhen daviszhen Awaiting requested review from daviszhen daviszhen is a code owner

@qingxinhome qingxinhome Awaiting requested review from qingxinhome qingxinhome is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
size/XL Denotes a PR that changes [1000, 1999] lines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@triump2020 @matrix-meow