Skip to content

[fix](case) catch exception to figure out failures - #54282

Merged
dataroaring merged 1 commit into
apache:masterfrom
dataroaring:sc_compaction
Aug 4, 2025
Merged

[fix](case) catch exception to figure out failures#54282
dataroaring merged 1 commit into
apache:masterfrom
dataroaring:sc_compaction

Conversation

@dataroaring

@dataroaringdataroaring commented Aug 4, 2025

Copy link
Copy Markdown
Contributor

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

@dataroaring

dataroaring commented Aug 4, 2025

Copy link
Copy Markdown
ContributorAuthor

run buildall

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Aug 4, 2025
@github-actions

github-actionsBot commented Aug 4, 2025

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actionsBot commented Aug 4, 2025

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

liaoxin01
liaoxin01 approved these changes Aug 4, 2025
View reviewed changes

@liaoxin01liaoxin01 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@dataroaring
dataroaring merged commit bbee496 into apache:masterAug 4, 2025
30 of 32 checks passed
github-actionsBot pushed a commit that referenced this pull request Aug 4, 2025
@dataroaringYour Name
[fix](case) catch exception to figure out failures (#54282)
8420e64
@github-actionsgithub-actionsBot mentioned this pull request Aug 4, 2025
Merged
github-actionsBot pushed a commit that referenced this pull request Aug 4, 2025
@dataroaringYour Name
[fix](case) catch exception to figure out failures (#54282)
c1ac691
@github-actionsgithub-actionsBot mentioned this pull request Aug 4, 2025
Merged
dataroaring added a commit that referenced this pull request Aug 4, 2025
@github-actions@dataroaring
branch-3.0: [fix](case) catch exception to figure out failures#54282(…
0a29560
…#54298)
Cherry-picked from #54282
Co-authored-by: Yongqiang YANG <yangyongqiang@selectdb.com>
morrySnow pushed a commit that referenced this pull request Aug 5, 2025
@github-actions@dataroaring
branch-3.1: [fix](case) catch exception to figure out failures#54282(…
37e28f8
…#54299)
Cherry-picked from #54282
Co-authored-by: Yongqiang YANG <yangyongqiang@selectdb.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@luwei16luwei16luwei16 approved these changes
@liaoxin01liaoxin01liaoxin01 approved these changes

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.dev/3.0.7-mergeddev/3.1.0-mergedp0_testreviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dataroaring@luwei16@liaoxin01@morrySnow