Skip to content

Commit

Permalink
Fix broken build caused by concurrent PR merges, a SpotBugs error
Browse files Browse the repository at this point in the history
- apache#9630 and apache#9329 were merged concurrently which caused the master branch
  to break

- fixes SpotBugs error
"Medium: org.apache.pulsar.client.impl.ConsumerBase.NONE_KEY should be
package protected [org.apache.pulsar.client.impl.ConsumerBase]
At ConsumerBase.java:[line 887] MS_PKGPROTECT"
  • Loading branch information
lhotari committed Feb 22, 2021
1 parent 65fed8b commit b378192
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -884,7 +884,7 @@ protected void callMessageListener(Message<T> msg) {
}
}

protected static final byte[] NONE_KEY = "NONE_KEY".getBytes(StandardCharsets.UTF_8);
static final byte[] NONE_KEY = "NONE_KEY".getBytes(StandardCharsets.UTF_8);
protected byte[] peekMessageKey(Message<T> msg) {
byte[] key = NONE_KEY;
if (msg.hasKey()) {
Expand Down

0 comments on commit b378192

Please sign in to comment.