[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-18 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16133765#comment-16133765 ] Hadoop QA commented on HBASE-18621: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vo

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16134011#comment-16134011 ] Hadoop QA commented on HBASE-18621: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vo

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-19 Thread Reid Chan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16134028#comment-16134028 ] Reid Chan commented on HBASE-18621: --- It's abnormal, trigger QA again. > Refactor Clust

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-19 Thread Reid Chan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16134158#comment-16134158 ] Reid Chan commented on HBASE-18621: --- There is conflict with [HBASE-18634|https://issue

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-22 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16136699#comment-16136699 ] Hadoop QA commented on HBASE-18621: --- | (/) *{color:green}+1 overall{color}* | \\ \\ ||

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-22 Thread Reid Chan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16136709#comment-16136709 ] Reid Chan commented on HBASE-18621: --- ping [~chia7712] [~mdrob] if you have time to rev

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-22 Thread Mike Drob (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16136847#comment-16136847 ] Mike Drob commented on HBASE-18621: --- {code} + * ClusterStatus status = admin.getCluster

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-22 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16136878#comment-16136878 ] Chia-Ping Tsai commented on HBASE-18621: I like Mike's idea. > Refactor ClusterO

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-22 Thread Reid Chan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16136876#comment-16136876 ] Reid Chan commented on HBASE-18621: --- Thanks [~mdrob], would you mind writing a piece of

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-22 Thread Reid Chan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16136894#comment-16136894 ] Reid Chan commented on HBASE-18621: --- I think i get it, let me try again. > Refactor Cl

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-22 Thread Mike Drob (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16136903#comment-16136903 ] Mike Drob commented on HBASE-18621: --- Sure... {code} enum Status { // this doesn't ch

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-22 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16137032#comment-16137032 ] Chia-Ping Tsai commented on HBASE-18621: Ditto for the ClusterStatus.proto ? Use

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-22 Thread Mike Drob (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16137131#comment-16137131 ] Mike Drob commented on HBASE-18621: --- I don't know enough about protos to say whether th

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-22 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16137212#comment-16137212 ] Chia-Ping Tsai commented on HBASE-18621: The changes looks like this. {code} enum

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-28 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16143606#comment-16143606 ] Chia-Ping Tsai commented on HBASE-18621: There is a related issue here. Why do we

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-28 Thread Reid Chan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16143628#comment-16143628 ] Reid Chan commented on HBASE-18621: --- I agree, and that's why i marked it private direct

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16143723#comment-16143723 ] Hadoop QA commented on HBASE-18621: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vo

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-28 Thread Reid Chan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16143838#comment-16143838 ] Reid Chan commented on HBASE-18621: --- ping [~chia7712] [~mdrob] for a new review, thanks

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16144162#comment-16144162 ] Hadoop QA commented on HBASE-18621: --- | (/) *{color:green}+1 overall{color}* | \\ \\ ||

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-29 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16145026#comment-16145026 ] Chia-Ping Tsai commented on HBASE-18621: Could we avoid the "ClusterStatus.Status

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-29 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16145455#comment-16145455 ] Hadoop QA commented on HBASE-18621: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vo

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-29 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16145890#comment-16145890 ] Hadoop QA commented on HBASE-18621: --- | (/) *{color:green}+1 overall{color}* | \\ \\ ||

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-30 Thread Reid Chan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16147336#comment-16147336 ] Reid Chan commented on HBASE-18621: --- Any other comments? [~chia7712] > Refactor Cluste

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-30 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16148403#comment-16148403 ] Hadoop QA commented on HBASE-18621: --- | (/) *{color:green}+1 overall{color}* | \\ \\ ||

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-30 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16148513#comment-16148513 ] Chia-Ping Tsai commented on HBASE-18621: we need javadoc for getClusterStatus().

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-31 Thread Reid Chan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16149051#comment-16149051 ] Reid Chan commented on HBASE-18621: --- QA is a bit long... > Refactor ClusterOptions bef

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-08-31 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16149128#comment-16149128 ] Chia-Ping Tsai commented on HBASE-18621: Recently, tests are mainly time out. We

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-09-01 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16150170#comment-16150170 ] Hadoop QA commented on HBASE-18621: --- | (/) *{color:green}+1 overall{color}* | \\ \\ ||

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-09-01 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16150354#comment-16150354 ] Chia-Ping Tsai commented on HBASE-18621: LGTM. [~mdrob] Any suggestions? > Refac

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-09-04 Thread Reid Chan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16152424#comment-16152424 ] Reid Chan commented on HBASE-18621: --- thank you [~chia7712] i will continue HBASE-18609

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-09-05 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16153746#comment-16153746 ] Chia-Ping Tsai commented on HBASE-18621: The fields in ClusterStatus may be null,

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-09-06 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16154921#comment-16154921 ] Hadoop QA commented on HBASE-18621: --- | (/) *{color:green}+1 overall{color}* | \\ \\ ||

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-09-06 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16155192#comment-16155192 ] Chia-Ping Tsai commented on HBASE-18621: Would you please add the tests for hashC

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-09-07 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16158157#comment-16158157 ] Hadoop QA commented on HBASE-18621: --- | (/) *{color:green}+1 overall{color}* | \\ \\ ||

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-09-08 Thread Chia-Ping Tsai (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16158370#comment-16158370 ] Chia-Ping Tsai commented on HBASE-18621: +1. Will commit it tomorrow if no object

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-09-08 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16159565#comment-16159565 ] Hudson commented on HBASE-18621: SUCCESS: Integrated in Jenkins build HBase-Trunk_matrix

[jira] [Commented] (HBASE-18621) Refactor ClusterOptions before applying to code base

2017-09-08 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-18621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16159574#comment-16159574 ] Hudson commented on HBASE-18621: FAILURE: Integrated in Jenkins build HBase-2.0 #480 (See