-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Fixes #1420 #1422
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
Fixes #1420 #1422
Conversation
@@ -281,4 +281,10 @@ | |||
* @param predicate the predicate | |||
*/ | |||
void flushChannelPoolPartitions(Predicate<Object> predicate); | |||
|
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.
Please format and remove spaces
@@ -276,4 +276,9 @@ protected BoundRequestBuilder requestBuilder(String method, String url) { | |||
protected BoundRequestBuilder requestBuilder(Request prototype) { | |||
return new BoundRequestBuilder(this, prototype).setSignatureCalculator(signatureCalculator); | |||
} | |||
|
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.
Please format and remove spaces
@@ -134,4 +134,9 @@ public ClientStats getClientStats() { | |||
public void flushChannelPoolPartitions(Predicate<Object> predicate) { | |||
throw new UnsupportedOperationException(); | |||
} | |||
|
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.
Please format and remove spaces
@@ -138,4 +138,9 @@ public ClientStats getClientStats() { | |||
public void flushChannelPoolPartitions(Predicate<Object> predicate) { | |||
throw new UnsupportedOperationException(); | |||
} | |||
|
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.
Please format and remove spaces
@slandelle What is your formatting preference? Note that the above notes are pieces of code that I did not touch. |
Of course you did. Those are all the places you added new methods. When you hit enter to jump line, your IDE doesn't create an empty line, but one with as many white spaces as the previous cursor position. Those are the ones I ask you to remove to I don't have to reformat after merging your PR. |
@slandelle My bad! 😭 Darn non-printable characters. The extra spaces should no longer be there. |
Thanks :) |
@slandelle Please review :)