[
https://issues.apache.org/jira/browse/HBASE-4969?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13165482#comment-13165482
]
Phabricator commented on HBASE-4969:
------------------------------------
stack has commented on the revision "HBASE-4969 [jira] tautology in
HRegionInfo.readFields".
@Nicolas
That made a big difference (I'll doc this in our book after I get it to
work). After installing cert which the tool guided me through nicely, I got to
this:
Exception:
ERR-CONDUIT-CORE: Bogus ID provided to load().
(Run with --trace for a full exception trace.)
h-24-30:hbase stack$ arc amend --trace --revision D669
Loading phutil library 'arclib' from
'/Users/Stack/checkouts/hbase/.arc_jira_lib'...
>>> [0] <conduit> conduit.connect()
<<< [0] <conduit> 418,950 us
>>> [1] <exec> $ (cd '/Users/Stack/checkouts/hbase'; git rev-parse
--show-cdup)
<<< [1] <exec> 10,368 us
>>> [2] <exec> $ (cd '/Users/Stack/checkouts/hbase/'; git rev-parse --verify
HEAD^)
<<< [2] <exec> 15,268 us
>>> [3] <exec> $ (cd '/Users/Stack/checkouts/hbase/'; git diff --no-ext-diff
--no-textconv --raw 'HEAD^' --)
>>> [4] <exec> $ (cd '/Users/Stack/checkouts/hbase/'; git diff --no-ext-diff
--no-textconv --raw HEAD --)
>>> [5] <exec> $ (cd '/Users/Stack/checkouts/hbase/'; git ls-files --others
--exclude-standard)
>>> [6] <exec> $ (cd '/Users/Stack/checkouts/hbase/'; git ls-files -m)
<<< [6] <exec> 62,638 us
<<< [3] <exec> 67,122 us
<<< [5] <exec> 65,706 us
<<< [4] <exec> 75,585 us
>>> [7] <conduit> differential.getcommitmessage()
Fatal error: Uncaught exception 'ConduitClientException' with message
'ERR-CONDUIT-CORE: Bogus ID provided to load().' in
/Users/Stack/checkouts/libphutil/src/conduit/client/ConduitFuture.php:54
Stack trace:
#0 /Users/Stack/checkouts/libphutil/src/future/proxy/FutureProxy.php(62):
ConduitFuture->didReceiveResult(Array)
#1 /Users/Stack/checkouts/libphutil/src/future/proxy/FutureProxy.php(39):
FutureProxy->getResult()
#2 /Users/Stack/checkouts/libphutil/src/conduit/client/ConduitClient.php(52):
FutureProxy->resolve()
#3
/Users/Stack/checkouts/arcanist/src/workflow/amend/ArcanistAmendWorkflow.php(125):
ConduitClient->callMethodSynchronous('differential.ge...', Array)
#4 /Users/Stack/checkouts/arcanist/scripts/arcanist.php(257):
ArcanistAmendWorkflow->run()
#5 {main}
thrown in
/Users/Stack/checkouts/libphutil/src/conduit/client/ConduitFuture.php on line 54
REVISION DETAIL
https://reviews.facebook.net/D669
> tautology in HRegionInfo.readFields
> -----------------------------------
>
> Key: HBASE-4969
> URL: https://issues.apache.org/jira/browse/HBASE-4969
> Project: HBase
> Issue Type: Bug
> Reporter: Prakash Khemani
> Assignee: Prakash Khemani
> Attachments: HBASE-4969.D669.1.patch
>
>
> In HRegionInfo.readFields() the following looks wrong to me
> } else if (getVersion() == VERSION) {
> it is always true.
> Should it have been
> } else if (getVersion() == version) {
> version is a local variable where the deserialized-version is stored.
> (I am struggling with another issue where after applying some patches -
> including "HBASE-4388 Second start after migration from 90 to trunk crashes"
> my version of hbase-92 HRegionInfo.readFields() tries to find HTD in
> HRegionInfo and fails)
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
