You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2018/02/20 20:21:12 UTC

[GitHub] sijie opened a new pull request #1260: Add JsonIgnore to a few fields in configuration objects

sijie opened a new pull request #1260: Add JsonIgnore to a few fields in configuration objects
URL: https://github.com/apache/incubator-pulsar/pull/1260
 
 
   *Problem*
   
   We have seen NPE thrown when `ConsumerStats` tries to dump the configuration object with a customized ConsumerEventsListener.
   The problem came from initialization sequence and when jackson tried to serialize the customized ConsumerEventsListener,
   it fails with NPE.
   
   In general, those customized implementation such as `MessageRouter`, `MessageListener` and `ConsumerEventsListener` are not
   really needed to serialize as part of json.
   
   *Solution*
   
   Add @JsonIgnore to those fields to bypass json serialization

----------------------------------------------------------------
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