[kudu-CR] hms tools: do not require HMS configuration flags

2018-08-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 14: Code-Review+2 -- To view, visit

[kudu-CR] hms tools: do not require HMS configuration flags

2018-08-01 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 14: (1 comment)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-08-01 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11036 to look at the new patch set (#14). Change subject: hms tools: do not require HMS configuration flags

[kudu-CR] hms tools: do not require HMS configuration flags

2018-08-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 13: Code-Review+2 (1 comment)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 13: Code-Review+2 (1 comment)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 13: (2 comments) > Patch Set 12: > > (2 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11036 to look at the new patch set (#13). Change subject: hms tools: do not require HMS configuration flags

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 12: (2 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 12: (2 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 12: (2 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11036 to look at the new patch set (#12). Change subject: hms tools: do not require HMS configuration flags

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 11: Code-Review+2 -- To view, visit

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 11: (6 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11036 to look at the new patch set (#11). Change subject: hms tools: do not require HMS configuration flags

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 10: (6 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Dan Burkert (Code Review)
Dan Burkert has removed a vote on this change. Change subject: hms tools: do not require HMS configuration flags .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11036 To

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-31 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 10: Verified+1 unrelated flake -- To view, visit

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-27 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 7: > Patch Set 6: > > One thing I noticed today: when

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-26 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 6: One thing I noticed today: when using the hms check

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-25 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11036 to look at the new patch set (#5). Change subject: hms tools: do not require HMS configuration flags

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11036 to look at the new patch set (#4). Change subject: hms tools: do not require HMS configuration flags

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 3: Code-Review+1 (1 comment)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 2: (2 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11036 to look at the new patch set (#3). Change subject: hms tools: do not require HMS configuration flags

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 2: (3 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 2: (8 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11036 to look at the new patch set (#2). Change subject: hms tools: do not require HMS configuration flags

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 1: (7 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11036 ) Change subject: hms tools: do not require HMS configuration flags .. Patch Set 1: (2 comments)

[kudu-CR] hms tools: do not require HMS configuration flags

2018-07-24 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Hao Hao, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11036 to review the following change. Change subject: hms tools: do not require HMS configuration flags .. hms