-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
KAFKA-5893: ResetIntegrationTest fails #3859
Conversation
mjsax
commented
Sep 14, 2017
- improve stderr output for better debugging
- improve stderr output for better debugging
This is WIP. I hope the build fails. We got quite some failing builds but the output only says:
Ie, the exception we catch has no error message. Printing the whole stack trace should shed some light on this. |
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 @mjsax LGTM
@mjsax, Did you want me to merge this? You said it is a WIP in your comment, so i'm unsure? |
Well. This does not fix the issue that's why it's WIP -- we can merge to figure out the root cause (the change makes sense anyway). But we should not close the JIRA as resolved (-> WIP). |
@@ -124,6 +124,7 @@ public int run(final String[] args, final Properties config) { | |||
} catch (final Throwable e) { | |||
exitCode = EXIT_CODE_ERROR; | |||
System.err.println("ERROR: " + e.getMessage()); | |||
e.printStackTrace(System.err); |
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.
In line 126 above, maybe we can just call e.toString
which will include the message string if it contains? otherwise LGTM
Is this failure easily reproducible locally, or only on Jenkins? I'd prefer to merge a PR that fixes the issue altogether with logging improvements. We can use jenkins bots to continuously trigger builds until it fails. |
@guozhangwang Updated this PR. I could not reproduce it locally, so I guess we need to merge this to get more insight why it's failing. |
retest this please |
retest this please. |
Merged to trunk |