On 08/26/2014 02:03 AM, Jonathan Nieder wrote: > Jonathan Nieder wrote: > [...] >> I've having trouble keeping track of how patches change, which patches >> have been reviewed and which haven't, unaddressed comments, and so on, >> so as an experiment I've pushed this part of the series to the Gerrit >> server at >> >> https://code-review.googlesource.com/#/q/topic:ref-transaction-1 > > Outcome of the experiment: patches gained some minor changes and then > [...] > > I found the web UI helpful in seeing how each patch evolved since I > last looked at it. Interdiff relative to the version in pu is below. > I'm still hoping for a tweak in response to a minor comment and then I > can put up a copy of the updated series to pull. Thanks for organizing this "experiment". I was one of the guinea pigs :-) I have wanted to review more of Ronnie's patches (actually, all of them!) but have been overwhelmed by the number of iterations and the number of patch series flying around in parallel. I was also interested to try out Gerrit, which I haven't used before. So I took up Jonathan's invitation and reviewed the first patch series in Gerrit. Here are some of my first impressions. * Overall, I found it easier to review commits in Gerrit than on the mailing list, especially a long patch series like this one that has seen so much flux. It was easier to see the comments from all reviewers that apply to a patch, which is difficult on the mailing list when comments are scattered over the many iterations of the patch series. It was easier to incrementally increase the context around a patch. It was easy to use the copy-paste commands provided in the "download" menu to fetch the commit that I was reviewing into my local Git repository, and from there to build it or investigate it using other tools. * The Gerrit interface is very busy. It was somewhat overwhelming to me as a beginner. On the other hand, the help menus ("?" key) are good and the keyboard shortcuts are convenient. I didn't have to read much documentation to get started doing review in Gerrit, at least at a basic level. * During two of my big Gerrit sessions the website was very responsive and pleasant to use. During the third, it was terribly slow, like 5 - 15 seconds per page update. If I had only experienced the slow behavior, I would have rejected Gerrit immediately. I hope that the slow behavior was a rare anomaly. * Gerrit sends out an endless flood of emails that mostly seem pretty useless to me. I wish it weren't so chatty and that its emails were better organized. * At one point a back-and-forth in a line comment grew into a more general issue that was more appropriate for the mailing list. The transition from Gerrit to mailing list was a bit awkward. So, overall I found it pleasant and efficient to review patches in Gerrit. I would welcome more such "experiments". It would have been even better if Gerrit would generate more useful notification emails. Michael -- Michael Haggerty mhagger@xxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html