-
Notifications
You must be signed in to change notification settings - Fork 394
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.
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
[#5152] fix(oceanbase-catalog): OB catalog UTs failed on Mac #5178
Conversation
Can you please fix the style issue @featherchen ? |
@jerryshao Sure, but actually the OB catalog UTs will still fail after these changes. |
@mchades would you please help on this PR? |
you can find the container logs in |
12feb14
to
988a1a3
Compare
I see that CI has passed, can you successfully run the OB catalog container tests on your local Mac? |
988a1a3
to
be4e420
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have manually tested this locally on my Mac, and it works. I am going to merge this PR into the main branch.
Thank you for your contributions!
…pache#5178) ### What changes were proposed in this pull request? The max user processes in the OB catalog unit tests should be larger than 120000 ### Why are the changes needed? Fix: apache#5152 ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? ./gradlew :catalogs:catalog-jdbc-oceanbase:test -PskipDockerTests=false
What changes were proposed in this pull request?
The max user processes in the OB catalog unit tests should be larger than 120000
Why are the changes needed?
Fix: #5152
Does this PR introduce any user-facing change?
No
How was this patch tested?
./gradlew :catalogs:catalog-jdbc-oceanbase:test -PskipDockerTests=false