rdhabalia opened a new pull request #6782:
URL: https://github.com/apache/pulsar/pull/6782


   ### Motivation
   In #4284, made 
[Authentication](https://github.com/apache/pulsar/blob/master/pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Authentication.java#L33)
 a transient under ClientConfigurationData. `Authentication` is a serializable 
class because ClientConfig is used in Storm and Spark adapter to pass client 
configuration and Storm serializes and deserializes spout and bolt while 
executing them in topology. Now, after making `Authentication` transient 
variable storm always deserializes it as null and authentication fails. Also 
`Authentication` is a serializable class so, any auth-implementation must be 
serializable.
   
   ### Modification
   Keep Authentication param serializable.
   
   ### Result
   It fixes pulsar-storm with authentication enabled.
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to