Jonathan Hsieh created HBASE-16668:

             Summary: Admin class should have a synchronous Admin#mergeRegions* 
                 Key: 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 

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 

This message was sent by Atlassian JIRA

Reply via email to