Skip to content

[feat](cloud) Support fdb multi version clients #59360

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
gavinchou merged 1 commit into from
Dec 26, 2025
Merged

[feat](cloud) Support fdb multi version clients #59360

gavinchou merged 1 commit into from
Dec 26, 2025
+352 −67

Conversation

@w41ter
Copy link
Contributor

@w41ter w41ter commented Dec 25, 2025

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

@w41ter w41ter added the cloud label Dec 25, 2025
@hello-stephen
Copy link
Contributor

hello-stephen commented Dec 25, 2025

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@w41ter
Copy link
Contributor Author

w41ter commented Dec 25, 2025

run buildall

2 similar comments
@w41ter
Copy link
Contributor Author

w41ter commented Dec 25, 2025

run buildall

@w41ter
Copy link
Contributor Author

w41ter commented Dec 25, 2025

run buildall

gavinchou
gavinchou previously approved these changes Dec 25, 2025
View reviewed changes
@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Dec 25, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 25, 2025

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

@github-actions
Copy link
Contributor

github-actions bot commented Dec 25, 2025

PR approved by anyone and no changes requested.

@w41ter
Copy link
Contributor Author

w41ter commented Dec 25, 2025

run buildall

@w41ter w41ter force-pushed the cloud/improve/fdb_multi_version_client branch from 51d6737 to 7c18eee Compare December 26, 2025 03:44
@w41ter
Copy link
Contributor Author

w41ter commented Dec 26, 2025

run compile

1 similar comment
@w41ter
Copy link
Contributor Author

w41ter commented Dec 26, 2025

run compile

@w41ter w41ter force-pushed the cloud/improve/fdb_multi_version_client branch from 3ecb6b1 to 7450998 Compare December 26, 2025 04:34
@w41ter
Copy link
Contributor Author

w41ter commented Dec 26, 2025

run buildall

@w41ter w41ter force-pushed the cloud/improve/fdb_multi_version_client branch from 7450998 to 79e0d02 Compare December 26, 2025 06:00
@w41ter
Copy link
Contributor Author

w41ter commented Dec 26, 2025

run buildall

@w41ter w41ter force-pushed the cloud/improve/fdb_multi_version_client branch from 79e0d02 to 00a7b4d Compare December 26, 2025 06:14
@w41ter
Copy link
Contributor Author

w41ter commented Dec 26, 2025

run buildall

@w41ter w41ter force-pushed the cloud/improve/fdb_multi_version_client branch from 00a7b4d to 85a90b8 Compare December 26, 2025 06:24
@w41ter
Copy link
Contributor Author

w41ter commented Dec 26, 2025

run buildall

@w41ter w41ter force-pushed the cloud/improve/fdb_multi_version_client branch from 85a90b8 to 5ca27c1 Compare December 26, 2025 06:39
@w41ter
Copy link
Contributor Author

w41ter commented Dec 26, 2025

run buildall

@hello-stephen
Copy link
Contributor

hello-stephen commented Dec 26, 2025

Cloud UT Coverage Report

Increment line coverage 43.90% (18/41) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 79.52% (1771/2227)
Line Coverage 64.81% (31303/48299)
Region Coverage 65.34% (15570/23831)
Branch Coverage 55.94% (8277/14796)

@gavinchou gavinchou merged commit cf57972 into apache:master Dec 26, 2025
26 of 27 checks passed
@w41ter w41ter deleted the cloud/improve/fdb_multi_version_client branch December 26, 2025 09:04
@w41ter w41ter mentioned this pull request Dec 26, 2025
Open
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gavinchou gavinchou gavinchou approved these changes

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

+1 more reviewer

@koarz koarz koarz approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by one committer. cloud dev/4.0.x dev/4.0.x-conflict reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@w41ter @hello-stephen @gavinchou @koarz