[ 
https://issues.apache.org/jira/browse/HBASE-3577?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12999908#comment-12999908
 ] 

Kazuki Ohta commented on HBASE-3577:
------------------------------------

your plan is acceptable to me. Then, I'll change the interface like below.

Just replace the TServerAddress with just Text in "host:port" format.

{code}
struct TRegionLocation {
  1:TRegionInfo regionInfo,
  2:Text serverAddress
}
{code}
{code}
map<TRegionInfo, Text> getRegionsInfo(...)
{code}


> enables Thrift client to get the Region location
> ------------------------------------------------
>
>                 Key: HBASE-3577
>                 URL: https://issues.apache.org/jira/browse/HBASE-3577
>             Project: HBase
>          Issue Type: Improvement
>          Components: thrift
>            Reporter: Kazuki Ohta
>         Attachments: HBASE3577-1.patch
>
>
> The current thrift interface has the getTableRegions() interface like below.
> {code}
>   list<TRegionInfo> getTableRegions(
>     /** table name */
>     1:Text tableName)
>     throws (1:IOError io)
> {code}
> {code}
> struct TRegionInfo {
>   1:Text startKey,
>   2:Text endKey,
>   3:i64 id,
>   4:Text name,
>   5:byte version
> }
> {code}
> But the method don't have the region location information (where the region 
> is located).
> I want to add the Thrift interfaces like below in HTable.java.
> {code}
> public Map<HRegionInfo, HServerAddress> getRegionsInfo() throws IOException
> {code}
> {code}
> public HRegionLocation getRegionLocation(final String row)
> {code}

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to