Github user revans2 commented on the issue:

    https://github.com/apache/storm/pull/2109
  
    @erikdw and @srishtyagrawal I know that we are still working through issues 
with checkstyle being new, but I get a little nervous with making too many 
changes to the "standard".
    
    The more changes we make to our style the harder it is to explain to people 
what that style is and the harder it is to setup an IDE to conform with it.  As 
it is right now we can say (and we should update the docs for this)  that we 
conform to the google standard except that we use 4 spaces for indentation and 
have a max line length of 140.  But if we start adding in more changes it gets 
harder to explain and we end up being like HBase where some impossible to read 
file is the documentation.  If it is not too much of a problem would we rename 
{{taskID}} to {{taskId}}?


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to