-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
MINOR: Fix a toString npe in tableProcessorNode #6807
Conversation
Worth adding a unit test. |
Fair enough |
c12512e
to
fc7f17a
Compare
checkstyle seems not happy. |
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.
Otherwise, lgtm.
Also, the target branch should be |
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 @rodesai! LGTM modulo checkstyle
fc7f17a
to
11e7518
Compare
JDK 11 build passed, JDK 8 build failed with flaky and unrelated |
This gets hit when debug logging is enabled. Reviewers: Bill Bejeck <bill@confluent.io>, Guozhang Wang <wangguoz@gmail.com>
Merged to trunk and 2.3 branches. |
This gets hit when debug logging is enabled. Reviewers: Bill Bejeck <bill@confluent.io>, Guozhang Wang <wangguoz@gmail.com>
This gets hit when debug logging is enabled. Reviewers: Bill Bejeck <bill@confluent.io>, Guozhang Wang <wangguoz@gmail.com>
Fix a toString npe in tableProcessorNode. This gets hit when we run
KSQL with debug logging enabled.