hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Hsieh (JIRA)" <j...@apache.org>
Subject [jira] [Created] (HBASE-16668) Admin class should have a synchronous Admin#mergeRegions* method
Date Wed, 21 Sep 2016 18:29:20 GMT
Jonathan Hsieh created HBASE-16668:

             Summary: Admin class should have a synchronous Admin#mergeRegions* method 
                 Key: HBASE-16668
                 URL: https://issues.apache.org/jira/browse/HBASE-16668
             Project: HBase
          Issue Type: Bug
          Components: hbase
    Affects Versions: 2.0.0
            Reporter: Jonathan Hsieh
             Fix For: 2.0.0

In trunk from HBASE-14552, we have deprecated {{void Admin#mergeRegions}} (in 1.x this was
an asynchronous call) and replaced it with {{Future<Void> Admin#mergeRegionsAsync}}
which is clearly async.    

This leaves us only with the async version.

We should have an easy way to make {{mergeRegions}} or an equivalant behave synchronously.

For normal java Futures, we could just call the future's {{get()}} method. Unforutnately,
the future this method returns doesn't follow java Future convention and throws Unimplemented
operation when a  plain {{get()}} is called and makes the api harder to use and read.  We
could make this future act more normally, and have the timeout throw an InterruptedException.

Alternately, we could expose a new method in {{Admin}} that behaves synchronously such as
{{HBaseAdmin#mergeRegionsSync}}. The caveat here is that we shouldn't use the name {{#mergeRegions}}
since it exists in 1.x with async semantics. 

This message was sent by Atlassian JIRA

View raw message