Github user revans2 commented on the pull request:

    https://github.com/apache/storm/pull/845#issuecomment-153511977
  
    @ptgoetz I don't consider this criticism and I'm not really trying to 
complain, I'm just trying to understand what requires an IP grant. 
    
    from http://incubator.apache.org/ip-clearance/index.html I find
    
    >  From time to time, an external codebase is brought into the ASF that is 
not a separate incubating project but still represents a substantial 
contribution that was not developed within the ASF's source control system and 
on our public mailing lists.
    
    and
    
    > The intent is to simply help to ensure, and record, that due diligence 
(Software Grant, CLA, Corp CLA, license and dependencies) has been paid to the 
incoming code, so that it can be merged into an existing project/module.
    
    Which seems a lot more vague than
    
    >   * Was the code developed in the open, using ASF infrastructure?
    >   * Was the Apache community aware of its existence from inception, or 
early on, such that it could contribute to it’s development?
    >   * Is the commit history intact such that all contributors can be 
identified?
    
    I can also see how these map from one to the other but not totally.  I am 
mostly confused about what is "a substantial contribution" which I am not 
disputing this is.  I am also confused about how putting a large pull request 
up differs from this?  Even though the core of this was developed in a separate 
repository anything and everything in this pull request is up for modification 
if the community feels that we need to.  The design document has been up for 
over a week, and the JIRA itself has been open since June that no one touched 
until now, but could have.  And most of the code has been touched in preparing 
it to work with Nimbus HA, specifically for this.  Lastly I don't see a reason 
to have the code contribution be attributed line by line, if we call out 
exactly who helped contribute to the patch.
    
    If you feel that we need an IP Grant for this I will make it happen, but 
going forward we the PMC need to have a discussion and clarify these questions, 
because to me leaving this up to a gut feeling is just going to put up too many 
barriers and potential frustration for developers.  Now I don't want line 
counts or anything like that, but something a lot less vague than "substantial"


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