+1 (non-binding)
On Wed, Oct 16, 2013 at 6:45 PM, Carl Steinbach <cwsteinb...@gmail.com>wrote: > +1 > > > On Wed, Oct 16, 2013 at 6:31 PM, Brock Noland <br...@cloudera.com> wrote: > > > +1 to remove the phabricator instructions from the wiki. > > > > On Wed, Oct 16, 2013 at 7:32 PM, Edward Capriolo <edlinuxg...@gmail.com> > > wrote: > > > Our wiki has instructions for posting to phabricator for code reviews. > > > https://cwiki.apache.org/confluence/display/Hive/PhabricatorCodeReview > > > > > > Phabricator now requires an external facebook account to review > patches, > > > and we have no technical support contact where phabricator is hosted. > It > > > also seems like some of the phabricator features are no longer working. > > > > > > Apache has a review board system many people are already using. > > > https://reviews.apache.org/account/login/?next_page=/dashboard/ > > > > > > This vote is to remove the phabricator instructions from the wiki. The > > > instructions will reference review board and that will be the only > system > > > that Hive supports for patch review process. > > > > > > +1 is a vote for removing the phabricator instructions from the wiki. > > > > > > Thank you, > > > Edward > > > > > > > > -- > > Apache MRUnit - Unit testing MapReduce - http://mrunit.apache.org > > >
