Ok, so it sounds like you are envisioning a "big bang" merge between the feature branch and master at some point. Not ideal in my mind, but maybe we need to be more pragmatic with this one.
Do we have other tasks (like these PRs) to complete before we are ready to consider the merge? I am just looking to help however I can in killing this feature branch; meaning I want your code in master, as soon as possible. :) On Wed, Sep 20, 2017 at 10:27 AM Otto Fowler <ottobackwa...@gmail.com> wrote: > So, were the functionality is not FB related, I have been doing PR’s > against master ( such as hdfs service ability to set permissions ). > I don’t think we have talked about the end game PR from feature to master, > I am don’t know how we would do multiple PRs to bring it down > once ‘accepted’. I think that approach needs to to be discussed and > agreed on. > > > On September 20, 2017 at 10:23:44, Nick Allen (n...@nickallen.org) wrote: > > So it sounds like these PRs do move us closer to bringing the two branches > together. But I think I am missing your high-level approach though. > > How are we going to get all of the functionality from the feature branch > into master? Are you going to bite-off small chunks of the feature branch > and introduce PRs against master? > > > > On Wed, Sep 20, 2017 at 9:23 AM Otto Fowler <ottobackwa...@gmail.com> > wrote: > >> Right now, these two PR’s are stacked, the UI depends on the BundleSystem >> changes. >> I would rather bring them down to feature before I do master integration, >> than integrate master >> into feature and then bring it out to the stacked PR’s. Simple as that. >> >> >> >> On September 20, 2017 at 08:35:09, Nick Allen (n...@nickallen.org) wrote: >> >> Hi Otto - >> >> What is the plan for bringing the feature branch and master together? Do >> these PRs move us closer to bringing the two branches together? >> >> Thanks >> >> >> >> On Wed, Sep 20, 2017 at 8:19 AM Otto Fowler <ottobackwa...@gmail.com> >> wrote: >> >> > Can I get a bump on https://github.com/apache/metron/pull/747 and >> > https://github.com/apache/metron/pull/761? >> > >> > The next time I take master is going to be a bit of integration work, >> and I >> > would like to unstack and get my stuff integrated first. >> > >> >>
