Github user nickwallen commented on the issue:

    https://github.com/apache/incubator-metron/pull/521
  
    > We would normally want to push format changes to a separate PR because 
it's hard to follow what has changed here,
    
    I don't know if we have a normal.  I've seen many instances go both ways 
during the course of the project.  I can certainly spend the time to split out 
formatting, if you need me to.  If you don't like the formatting, but want the 
Kerberos instructions that would be one reason for me to do so.
    
    I had a hard time understanding what we were trying to accomplish with each 
of the steps, which user the commands would be run under, etc which is why I 
split them under goal oriented headings.  It also helped me generalize the 
instructions to run in environments other than Full/Quick Dev.  
    
    
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to