-
Notifications
You must be signed in to change notification settings - Fork 153
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
verify dropwizard application log #367
base: main
Are you sure you want to change the base?
Conversation
polaris-service/src/test/java/org/apache/polaris/service/PolarisApplicationIntegrationTest.java
Outdated
Show resolved
Hide resolved
ready for review |
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.
Thanks @sullis for working on it. LGTM overall. Left minor comments and some questions. The only thing I'm not sure is whether we need to check the archived log. +1 if 2M is big enough for the log in the test.
polaris-service/src/test/java/org/apache/polaris/service/PolarisApplicationIntegrationTest.java
Outdated
Show resolved
Hide resolved
polaris-service/src/test/java/org/apache/polaris/service/PolarisApplicationIntegrationTest.java
Outdated
Show resolved
Hide resolved
polaris-service/src/test/java/org/apache/polaris/service/PolarisApplicationIntegrationTest.java
Outdated
Show resolved
Hide resolved
polaris-service/src/test/java/org/apache/polaris/service/PolarisApplicationIntegrationTest.java
Outdated
Show resolved
Hide resolved
d177de2
to
1b0d9f5
Compare
Ready for review. |
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.
Slick
waiting for GitHub Actions approval. |
Head branch was pushed to by a user without write access
GitHub workflows awaiting approval. |
workflows awaiting approval |
@sullis, looks like there is a conflict. It'd be nice if you can fix it, so that we can merge it. |
Head branch was pushed to by a user without write access
25cad6b
to
bfcae99
Compare
Workflows waiting for approval |
Description
add test to verify that there are zero errors in the dropwizard application log
Type of change
How Has This Been Tested?
Github Actions CI
Checklist: