Ethanlm commented on pull request #3377:
URL: https://github.com/apache/storm/pull/3377#issuecomment-800633946


   @wxy929629  Thanks for contributing!
   
   Can you please help me understand this problem better? 
   
   `sasl.mechanism` is set `GSSAPI` if not set. Why would it error when the 
security protocol `SASL_PLAINTEXT` is used? Does it only fail when the security 
protocol is set `SASL_PLAINTEXT`? What happens to other protocols like 
`PLAINTEXT`, `SASL_SSL` and `SSL`?
   
   Can you please explain more on what happened, by providing stack trace and 
logs if possible? Also the jira indicates it is tested on storm 1.2.2.  Can you 
provide the logs and stack trace from the testing from storm master branch?  
Although I understand this is mostly about kafka client setup, not very related 
to storm version, but it is still nice to see since this PR is based on storm 
master branch.  It will help reviewers and other storm users to understand this 
better.
   
   Also did you test this with Kafka-1.x and make sure it still works?
   
   
   Thanks


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