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

Ashish Singhi commented on HBASE-11393:
---------------------------------------

bq. Which spitter do you think is best?
The string tableCfs related api was deprecated in 0.99 as part of HBASE-11367 
hence we are good to remove it in 2.0.0. So for 2.0.0 we need not worry about 
this.

For other branches we can just assume that the table passed in tableCfs of 
string type belongs to default name space and set it in 
ZooKeeperProtos.TableCF. Please cross check the code once to see if that is how 
it works currently.
And in ruby script lets use non-deprecated method and update the usage to 
encourage users to use map of table name and CFs

[~enis], do you have any other better suggestions ?

> Replication TableCfs should be a PB object rather than a string
> ---------------------------------------------------------------
>
>                 Key: HBASE-11393
>                 URL: https://issues.apache.org/jira/browse/HBASE-11393
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Enis Soztutar
>             Fix For: 2.0.0
>
>         Attachments: HBASE-11393.patch, HBASE-11393_v1.patch, 
> HBASE-11393_v10.patch, HBASE-11393_v2.patch, HBASE-11393_v3.patch, 
> HBASE-11393_v4.patch, HBASE-11393_v5.patch, HBASE-11393_v6.patch, 
> HBASE-11393_v7.patch, HBASE-11393_v8.patch, HBASE-11393_v9.patch
>
>
> We concatenate the list of tables and column families in format  
> "table1:cf1,cf2;table2:cfA,cfB" in zookeeper for table-cf to replication peer 
> mapping. 
> This results in ugly parsing code. We should do this a PB object. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to