[jira] [Commented] (HBASE-17754) [C++] RawAsyncTable

2017-03-14 Thread Xiaobing Zhou (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15925165#comment-15925165 ] Xiaobing Zhou commented on HBASE-17754: --- Thanks [~enis] for the work. Some of my co

[jira] [Commented] (HBASE-17754) [C++] RawAsyncTable

2017-03-14 Thread Enis Soztutar (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15925251#comment-15925251 ] Enis Soztutar commented on HBASE-17754: --- bq. there are a couple of compile errors S

[jira] [Commented] (HBASE-17754) [C++] RawAsyncTable

2017-03-14 Thread Enis Soztutar (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15925268#comment-15925268 ] Enis Soztutar commented on HBASE-17754: --- bq. AsyncConnectionImpl::region_locator re

[jira] [Commented] (HBASE-17754) [C++] RawAsyncTable

2017-03-15 Thread Xiaobing Zhou (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15926857#comment-15926857 ] Xiaobing Zhou commented on HBASE-17754: --- Thanks for the new patch. # We should kee

[jira] [Commented] (HBASE-17754) [C++] RawAsyncTable

2017-03-15 Thread Enis Soztutar (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15927437#comment-15927437 ] Enis Soztutar commented on HBASE-17754: --- bq. We should keep the unit test. client-t

[jira] [Commented] (HBASE-17754) [C++] RawAsyncTable

2017-03-17 Thread Xiaobing Zhou (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15930467#comment-15930467 ] Xiaobing Zhou commented on HBASE-17754: --- Thanks for v3. It looks good to me. > [C+