Skip to content

Comments

Fix: make cpp binding build robust#6460

Merged
Xuanwo merged 5 commits intoapache:mainfrom
devillove084:fix/cpp_binding_build_upgrade
Aug 20, 2025
Merged

Fix: make cpp binding build robust#6460
Xuanwo merged 5 commits intoapache:mainfrom
devillove084:fix/cpp_binding_build_upgrade

Conversation

@devillove084
Copy link
Contributor

Which issue does this PR close?

Closes #6459

@devillove084 devillove084 requested a review from Xuanwo as a code owner July 31, 2025 17:10
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. releases-note/fix The PR fixes a bug or has a title that begins with "fix" labels Jul 31, 2025
@devillove084 devillove084 marked this pull request as draft July 31, 2025 17:10
@devillove084 devillove084 force-pushed the fix/cpp_binding_build_upgrade branch from c34f9ef to ff03771 Compare August 3, 2025 08:10
@devillove084 devillove084 force-pushed the fix/cpp_binding_build_upgrade branch from ff03771 to 07a4ef4 Compare August 3, 2025 08:18
@devillove084 devillove084 force-pushed the fix/cpp_binding_build_upgrade branch from 269de6b to 33eb45c Compare August 18, 2025 14:37
@devillove084 devillove084 marked this pull request as ready for review August 18, 2025 14:38
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Aug 18, 2025
@devillove084 devillove084 force-pushed the fix/cpp_binding_build_upgrade branch from 33eb45c to 8f092a1 Compare August 20, 2025 11:18
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Aug 20, 2025
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Aug 20, 2025
@devillove084
Copy link
Contributor Author

CC @Xuanwo

Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you!

@Xuanwo Xuanwo merged commit f0c74f6 into apache:main Aug 20, 2025
64 checks passed
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/fix The PR fixes a bug or has a title that begins with "fix" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Try make cpp binding build more robust

2 participants