Skip to content
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: Consistent terminal period in Errors.defaultExceptionMessage #3909

Merged
merged 1 commit into from
Feb 25, 2020

Conversation

tombentley
Copy link
Contributor

Some default messages ended with a period and some didn't, so make things consistent.

The contribution is my original work and I license the work to the project under the project's open source license.

@tombentley tombentley force-pushed the MINOR-Errors-punctuation branch from ebdaca4 to bb3f671 Compare February 25, 2020 11:31
@tombentley
Copy link
Contributor Author

@mimaison any chance you could reivew this one? It's trivial.

Copy link
Member

@mimaison mimaison left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

@mimaison mimaison merged commit 5216da3 into apache:trunk Feb 25, 2020
@hachikuji
Copy link

@mimaison This caused a failure in PlaintextAdminIntegrationTest.testElectPreferredLeaders. We probably should run the build even if the patch seems trivial.

@mimaison
Copy link
Member

@hachikuji My bad. I'll make sure I always run tests.

@tombentley
Copy link
Contributor Author

@hachikuji also my bad, sorry. According to https://cwiki.apache.org/confluence/display/KAFKA/Contributing+Code+Changes "The Jenkins automatic pull request builder will run unit and integration tests on your branch", I guess that's no longer true, and if I want to run the tests on Jenkins I need to request it specifically, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants