pulsar-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] jiazhai opened a new pull request #3034: Issue #2751: Add check to fix NPE
Date Wed, 21 Nov 2018 10:23:57 GMT
jiazhai opened a new pull request #3034: Issue #2751: Add check to fix NPE 
URL: https://github.com/apache/pulsar/pull/3034
 
 
   ### Motivation
   
   We may meet NPE like this: 
   ```
   java.lang.NullPointerException: null
   at org.apache.pulsar.common.api.proto.PulsarApi$KeyValue$Builder.setValue(PulsarApi.java:1923)
~[org.apache.pulsar-pulsar-common-2.1.1-incubating.jar:2.1.1-incubating]
   ```
   This is related to protobuf, it does not support null-able field directly.
   protocolbuffers/protobuf#1606
   
   In this fix we try to avoid this by add checking before this method is called.
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message