nickwallen commented on issue #1490: METRON-2169: Upgrade to Kafka 2.0.0 and 
Storm 1.2.1
URL: https://github.com/apache/metron/pull/1490#issuecomment-525416763
 
 
   Hey @merrimanr - Your test script mentions validating these changes under 
kerberos.  That might not be necessary based on your changes in this PR.  
   
   I would suggest that we hold off on validating kerberos until we have all 
the upgrade work merged into the feature branch.  This makes sense to me 
because (1) we are working on a feature branch and (2) our code doesn't really 
*do* much with kerberos, it is just *provided* by the underlying platform.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to