+John Sichi as he did a lot of the work on getting Hive to use Phabricator.

Is there anything preventing people from using Review Board for patches now, 
except maybe convention?  As for applying patches, in either case, you can 
download the patch from the JIRA site.

To be clear, I am not trying to aggressively force Phabricator on Hive, these 
are just suggestions.

From: Carl Steinbach <c...@cloudera.com<mailto:c...@cloudera.com>>
Date: Tue, 10 Apr 2012 18:10:44 -0700
To: Microsoft Office User <kevinwilf...@fb.com<mailto:kevinwilf...@fb.com>>
Cc: "dev@hive.apache.org<mailto:dev@hive.apache.org>" 
<dev@hive.apache.org<mailto:dev@hive.apache.org>>
Subject: Re: Problems with Arc/Phabricator

Ok, I'll submit an issue. But then what? Who is going to fix it? Perhaps in the 
meantime we should just switch back to using reviewboard while we wait for Arc 
and Phabricator to mature as open source projects.


On Tue, Apr 10, 2012 at 5:58 PM, Kevin Wilfong 
<kevinwilf...@fb.com<mailto:kevinwilf...@fb.com>> wrote:
There's a list of ways to report issues/get help on the side bar of this
site http://phabricator.org/

It suggests #phabricator 
<irc://chat.freenode.net/phabricator<http://chat.freenode.net/phabricator>> as 
the best
place to get help with issues.

On 4/10/12 5:42 PM, "Carl Steinbach" 
<c...@cloudera.com<mailto:c...@cloudera.com>> wrote:

>Arc won't let me apply the patch for HIVE-2646. I've also noticed some
>other problems (e.g. missing TEST PLAN in checkin comments), and was
>wondering how we can get these problems resolved? Since most of this work
>was done at Facebook, who is the designated point of contact for
>Arc/Phabricator issues going forward?
>
>
>Here's the error I get when applying HIVE-2646:
>
>% arc patch D2133 --trace
>Loading phutil library 'arc_jira_lib' from
>'/Users/carl/Work/repos/hive-test/.arc_jira_lib'...
>>>> [0] <exec> $ (cd '/Users/carl/Work/repos/hive-test'; git rev-parse
>--show-cdup)
><<< [0] <exec> 13,599 us
>>>> [1] <conduit> differential.getdiff()
><<< [1] <conduit> 608,237 us
>>>> [2] <exec> $ (cd '/Users/carl/Work/repos/hive-test/'; git apply
>>>>--index
>--reject)
><<< [2] <exec> 13,178 us
>
>Fatal error: Uncaught exception 'CommandException' with message 'Command
>'(cd '/Users/carl/Work/repos/hive-test/'; git apply --index --reject)'
>failed with error #128:
>stdout:
>
>stderr:
><stdin>:10: trailing whitespace.
>
><stdin>:111: trailing whitespace.
>    <fileset dir="${build.ivy.lib.dir}/default" includes="*.jar"
>fatal: corrupt patch at line 609
>
>' in /Users/carl/.local/pkg/libphutil/src/future/exec/ExecFuture.php:346
>Stack trace:
>#0
>/Users/carl/.local/pkg/arcanist/src/workflow/patch/ArcanistPatchWorkflow.p
>hp(412):
>ExecFuture->resolvex()
>#1 /Users/carl/.local/pkg/arcanist/scripts/arcanist.php(266):
>ArcanistPatchWorkflow->run()
>#2 {main}
>  thrown in
>/Users/carl/.local/pkg/libphutil/src/future/exec/ExecFuture.php
>on line 346
>
>
>Thanks.
>
>Carl


Reply via email to