You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by mm...@apache.org on 2021/02/22 16:46:43 UTC
[pulsar] branch master updated: [BUILD] Master branch failed by
spotbugs error (#9669)
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 5755243 [BUILD] Master branch failed by spotbugs error (#9669)
5755243 is described below
commit 5755243907d735142c02831c3127ca02f973fc95
Author: Yong Zhang <zh...@gmail.com>
AuthorDate: Tue Feb 23 00:45:51 2021 +0800
[BUILD] Master branch failed by spotbugs error (#9669)
---
*Motivation*
Master branch failed by [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
---
.../src/main/java/org/apache/pulsar/client/impl/ConsumerBase.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerBase.java b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerBase.java
index 88ea82e..ddd5dff 100644
--- a/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerBase.java
+++ b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerBase.java
@@ -884,7 +884,7 @@ public abstract class ConsumerBase<T> extends HandlerState implements Consumer<T
}
}
- 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()) {