Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Hive, mail # dev - VOTE: Remove phabricator instructions from hive-development guide (wiki), officially only support Apache's review board.


Copy link to this message
-
Re: VOTE: Remove phabricator instructions from hive-development guide (wiki), officially only support Apache's review board.
Owen O'Malley 2013-10-17, 21:18
Ed,
  I didn't remember being unable to see revisions without a login. That is
uncool. I'll change my vote to +1.

-- Owen
On Wed, Oct 16, 2013 at 9:08 PM, Edward Capriolo <[EMAIL PROTECTED]>wrote:

> Owen,
> In your issues:
> https://issues.apache.org/jira/browse/HIVE-5567
>
> When I click this link:
> REVISION DETAIL
> https://reviews.facebook.net/D13479
>
> I am prompted for a password.
>
>
>
> On Wed, Oct 16, 2013 at 11:16 PM, Owen O'Malley <[EMAIL PROTECTED]
> >wrote:
>
> > -0
> >
> > I like phabricator, but it is a pain to setup. It doesn't require a fb
> > account, but clearly it isn't managed or supported by Apache.
> >
> > -- Owen
> >
> > > On Oct 16, 2013, at 17:32, Edward Capriolo <[EMAIL PROTECTED]>
> > 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
> >
>