Github user hmcl commented on the issue:

    https://github.com/apache/storm/pull/1649
  
    @harshach I can squash the commits, but I left these three commits on 
purpose. Each commit is a logical groping. One is docs, one is the default 
values, and one is the logging changes. They all have the same JIRA ticket, but 
the commit log messages suit the changes. The reason behind this choice is that 
it is easier to cherry pick only part of the changes to a given branch - if 
needed. I also believe that it makes it easier for an external person to follow 
what each patch addresses.
    
    I am OK with either approach. This squash is pretty simple. Let me know if 
you prefer that this is indeed squashed, and I will do so. Thanks!


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