Hi guys, Thanks for opening a [DISCUSS] thread.
I am -1 on the concept of using github as our code review tool, for a few reasons. The biggest reason is just the "split system" effect. I don't want to have to search a separate system to see the patch that would formerly have been attached to a JIRA. Sometimes I want to search for a particular comment that someone made. I can do this through JIRA full text search, but I can't do it through github. I would like the discussion of the JIRA, to appear on the JIRA, not some other place. I would like to be able to learn everything I need, just from watching the jira, and not have to scramble to other 3rd party websites. I don't think comments about the JIRA can always be cleanly separated into comments about the patch and comments about the idea. if I want to make meta-comments about an approach, where do they go? People will stop reading JIRA if all the action is on github. But github is apparently only for detailed patch comments. >From an interface point of view, I just don't like github's interface. I admit that this is somewhat of a matter of opinion, but it's how I feel. I don't like how each new patch version you post hides the comments from the previous version. It makes it difficult to follow the thread of the conversation because I have to keep clicking on "expand" arrows constantly, whereas with JIRA I could just read from top to bottom. I have found that projects that used github heavily, such as Spark, had problems with multiple PRs being opened for the same issue. The people didn't know about each others' PR. JIRA was often not updated with the results of discussions on github, and JIRA summaries for that project could be very misleading. They build some custom tooling to try to alleviate this problem, but it's a problem we just wouldn't have if we stuck with JIRA. If the concern is that we need a better code review tool, how about Crucible? That integrates with JIRA rather than being a separate system. best, Colin On Tue, Apr 21, 2015 at 11:53 AM, Jake Farrell <[email protected]> wrote: > We have that available also, when a pr is created we can launch a jenkins > run for a given project and jenkins will comment back on that pr when its > done with the status. Also for Phoenix I enabled it in the filter for the > pre-commit last night > > -Jake > > > > On Tue, Apr 21, 2015 at 2:41 PM, Nick Dimiduk <[email protected]> wrote: >> >> Jake -- >> >> While I'm making requests of Infra, it would be super-cool if the >> Pre-Commit bot could detect a PR the same way it detects a patch. Then we >> could have our pre-commit scrips pull down the change, and run it like a >> patch through the project's usual process. Better still would be if Apache >> could provide a template for projects to follow. In another thread, I'm >> adding pre commit for Phoenix, which is going to end up using a fork of >> HBase's.. which is itself a fork of Hadoop's. One base script that does most >> of the heavy lifting that we all can inherit from and extend with >> project-specific additions would be really helpful, especially for when I >> get around to adding pre-commit for HTrace too ;) >> >> -n >> >> On Tue, Apr 21, 2015 at 11:36 AM, Nick Dimiduk <[email protected]> >> wrote: >>> >>> Hi Colin, thanks for making this a DISCUSS thread ;) >>> >>> I'm not suggesting switching away from JIRA. What I want is to make it as >>> easy as possible for non-Apache folks to find and contribute to HTrace, >>> through the following: >>> (0) mirror our repo to the github.com/apache account >>> (1) allow folks to submit a PR against that account and have it arrive >>> meaningfully to the community. On other projects, I've seen this result in a >>> mail to dev@ >>> >>> After that it's up for discussion. I think PR's and RB are a better place >>> for deconstructing a large patch than comments on a JIRA. Github has the >>> advantage over apache RB of being the most visible code review tool out >>> there, so it benefits our project by participating. I also don't think it's >>> a big deal to review patches on PR. Just like with RB, it's external to JIRA >>> and that's okay, so long as it's clear that there's a non-JIRA resource >>> associated with the ticket. For initial contributions that originate on a >>> PR, it's obvious where to look for additional discussion. It's the exact >>> same as looking to RB or Fabricator for patch comments. It's up to the >>> contributor and the sponsor committed to be aware of these threads. >>> >>> I am absolutely NOT suggesting we replace JIRA with github issue tracker >>> or pushing commits that lack and associated JIRA. >>> >>> Thanks Jake for the docs on how Thrift integrates with GH, that's what I >>> was looking for. >>> >>> On Tue, Apr 21, 2015 at 11:23 AM, Jake Farrell <[email protected]> >>> wrote: >>>> >>>> Hey Colin >>>> I don't think there was any intention of switching away from jira, just >>>> enabling the github integrations for the project to accept patches from >>>> github. For Thrift we require that all github pull requests have a jira >>>> associated with them, for exactly as you point out, its hard to track >>>> everything in split systems >>>> >>>> -Jake >>>> >>>> On Tue, Apr 21, 2015 at 1:53 PM, Colin P. McCabe <[email protected]> >>>> wrote: >>>>> >>>>> Hi guys, >>>>> >>>>> I have an objection. In the past, I've found it frustrating to search >>>>> through github pull requests. There is no interface (like there is on >>>>> JIRA) to search using any kind of structured query language, and we >>>>> don't have the tools to track things by release, contributor, etc. >>>>> >>>>> If we start having some of our patch discussions on github, JIRA will >>>>> become a lot less useful. We might run into a situation like on Spark >>>>> where people open multiple pull requests for the same thing, not >>>>> knowing about each other. Or people have a discussion on JIRA, not >>>>> aware that a parallel discussion is going on on github. >>>>> >>>>> I think we should take more time to think this through. 1 hour is not >>>>> enough time to decide to switch away from JIRA :) >>>>> >>>>> best, >>>>> Colin >>>>> >>>>> On Tue, Apr 21, 2015 at 10:21 AM, Jake Farrell <[email protected]> >>>>> wrote: >>>>> > Once Github picks up the mirror i'll enable the remaining >>>>> > integrations >>>>> > steps so we will start getting notices on our dev@ list and can close >>>>> > our >>>>> > pull requests through commits. Here are some docs I did for Thrift >>>>> > that >>>>> > would be good to adopt or change for how people contribute or commit >>>>> > to >>>>> > HTrace. >>>>> > >>>>> > http://thrift.apache.org/docs/HowToContribute >>>>> > http://thrift.apache.org/docs/committers/HowToCommit >>>>> > >>>>> > If you have any other questions let me know >>>>> > >>>>> > -Jake >>>>> > >>>>> > >>>>> > >>>>> > >>>>> > >>>>> > On Tue, Apr 21, 2015 at 12:57 PM, Nick Dimiduk <[email protected]> >>>>> > wrote: >>>>> > >>>>> >> That was fast, thanks Jake :) >>>>> >> >>>>> >> What else do we need to to to get the fancy PR integration i've seen >>>>> >> in >>>>> >> other projects? I see there's a specific task type for that on INFRA >>>>> >> Jira. >>>>> >> Is there a doc for Apacheer-but-not-githubbers on what the workflow >>>>> >> looks >>>>> >> like? Or is it just read the Github docs on PR's? >>>>> >> >>>>> >> Thanks, >>>>> >> Nick >>>>> >> >>>>> >> On Tue, Apr 21, 2015 at 9:54 AM, Jake Farrell <[email protected]> >>>>> >> wrote: >>>>> >> >>>>> >>> Hey Eliott >>>>> >>> Great idea, I have setup the git.a.o mirror for us and will enable >>>>> >>> the >>>>> >>> Github integrations as soon as Github picks up the repo from >>>>> >>> git.a.o >>>>> >>> (usually within 24 hours) >>>>> >>> >>>>> >>> -Jake >>>>> >>> >>>>> >>> On Tue, Apr 21, 2015 at 12:46 PM, Elliott Clark <[email protected]> >>>>> >>> wrote: >>>>> >>> >>>>> >>> > That would be great. >>>>> >>> > >>>>> >>> > On Tue, Apr 21, 2015 at 9:34 AM, Nick Dimiduk >>>>> >>> > <[email protected]> >>>>> >>> wrote: >>>>> >>> > >>>>> >>> > > Do we have any kind of github integration setup? I can't even >>>>> >>> > > find a >>>>> >>> > mirror >>>>> >>> > > of HTrace on the apache account. I think we'll make it easier >>>>> >>> > > for >>>>> >>> folks >>>>> >>> > to >>>>> >>> > > contribute if they can send PR's. >>>>> >>> > > >>>>> >>> > > I'd like to open an INFRA ticket to get us setup with this >>>>> >>> integration. >>>>> >>> > Are >>>>> >>> > > there any objections? >>>>> >>> > > >>>>> >>> > > Thanks, >>>>> >>> > > Nick >>>>> >>> > > >>>>> >>> > >>>>> >>> >>>>> >> >>>>> >> >>>> >>>> >>> >> >
