[kudu-CR] [ranger] validate the JAR file path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15435 to look at the new patch set (#4). Change subject: [ranger] validate the JAR file path for the subprocess

[kudu-CR] [ranger] pass 'principal' and 'keytab' to the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15416 ) Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess .. Patch Set 6: (1 comment)

[kudu-CR] [ranger] pass 'principal' and 'keytab' to the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15416 to look at the new patch set (#6). Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess

[kudu-CR] [ranger] fix incorrect authz enforcement in Ranger authz provider

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15436 ) Change subject: [ranger] fix incorrect authz enforcement in Ranger authz provider .. Patch Set 3: (2 comments)

[kudu-CR] [ranger] fix incorrect authz enforcement in Ranger authz provider

2020-03-14 Thread Hao Hao (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15436 to look at the new patch set (#4). Change subject: [ranger] fix incorrect authz enforcement in Ranger authz provider

[kudu-CR] [ranger] pass 'principal' and 'keytab' to the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15416 ) Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess .. Patch Set 5: (1 comment)

[kudu-CR] [subprocess] copy subprocess JAR to be next to the master binary

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15438 ) Change subject: [subprocess] copy subprocess JAR to be next to the master binary .. Patch Set 3: (1 comment)

[kudu-CR] [subprocess] copy subprocess JAR to be next to the master binary

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [subprocess] copy subprocess JAR to be next to the master binary .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/15438 To

[kudu-CR] [ranger] parameterize the Java binary path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15415 ) Change subject: [ranger] parameterize the Java binary path for the subprocess .. Patch Set 6: Verified+1 Unrelated flaky test failure --

[kudu-CR] [ranger] parameterize the Java binary path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [ranger] parameterize the Java binary path for the subprocess .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/15415 To

[kudu-CR] KUDU-2780: create thread for auto-rebalancing

2020-03-14 Thread Hannah Nguyen (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14177 to look at the new patch set (#23). Change subject: KUDU-2780: create thread for auto-rebalancing

[kudu-CR] [ranger] fix incorrect authz enforcement in Ranger authz provider

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15436 ) Change subject: [ranger] fix incorrect authz enforcement in Ranger authz provider .. Patch Set 3: (1 comment)

[kudu-CR] [ranger] fix incorrect authz enforcement in Ranger authz provider

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15436 ) Change subject: [ranger] fix incorrect authz enforcement in Ranger authz provider .. Patch Set 3: (1 comment)

[kudu-CR] [subprocess] copy subprocess JAR to be next to the master binary

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15438 ) Change subject: [subprocess] copy subprocess JAR to be next to the master binary .. Patch Set 3: (1 comment)

[kudu-CR] [ranger] fix incorrect authz enforcement in Ranger authz provider

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15436 ) Change subject: [ranger] fix incorrect authz enforcement in Ranger authz provider .. Patch Set 3: (1 comment)

[kudu-CR] [ranger] pass 'principal' and 'keytab' to the subprocess

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15416 ) Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess .. Patch Set 5: (2 comments)

[kudu-CR] [ranger] pass 'principal' and 'keytab' to the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15416 to look at the new patch set (#5). Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess

[kudu-CR] [subprocess] copy subprocess JAR to be next to the master binary

2020-03-14 Thread Hao Hao (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15438 to look at the new patch set (#3). Change subject: [subprocess] copy subprocess JAR to be next to the master binary

[kudu-CR] [subprocess] copy subprocess JAR to be next to the master binary

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15438 ) Change subject: [subprocess] copy subprocess JAR to be next to the master binary .. Patch Set 2: > Patch Set 1: > > I think we can remove

[kudu-CR] KUDU-2780: create thread for auto-rebalancing

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14177 ) Change subject: KUDU-2780: create thread for auto-rebalancing .. Patch Set 22: (1 comment)

[kudu-CR] [subprocess] copy subprocess JAR to be next to the master binary

2020-03-14 Thread Hao Hao (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15438 to look at the new patch set (#2). Change subject: [subprocess] copy subprocess JAR to be next to the master binary

[kudu-CR] [ranger] fix incorrect authz enforcement in Ranger authz provider

2020-03-14 Thread Hao Hao (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15436 to look at the new patch set (#2). Change subject: [ranger] fix incorrect authz enforcement in Ranger authz provider

[kudu-CR] [ranger] pass 'principal' and 'keytab' to the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15416 ) Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess .. Patch Set 4: (4 comments)

[kudu-CR] [ranger] pass 'principal' and 'keytab' to the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15416 to look at the new patch set (#4). Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess

[kudu-CR] [ranger] fix incorrect authz enforcement in Ranger authz provider

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15436 ) Change subject: [ranger] fix incorrect authz enforcement in Ranger authz provider .. Patch Set 1: (2 comments)

[kudu-CR] [ranger] validate the JAR file path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15435 ) Change subject: [ranger] validate the JAR file path for the subprocess .. Patch Set 2: (1 comment)

[kudu-CR] [ranger] validate the JAR file path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15435 to look at the new patch set (#2). Change subject: [ranger] validate the JAR file path for the subprocess

[kudu-CR] [ranger] parameterize the Java binary path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15415 to look at the new patch set (#5). Change subject: [ranger] parameterize the Java binary path for the subprocess

[kudu-CR] [java] fix Kudu Ranger plugin when Ranger is Kerberized

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15414 ) Change subject: [java] fix Kudu Ranger plugin when Ranger is Kerberized .. Patch Set 4: (2 comments)

[kudu-CR] [java] fix Kudu Ranger plugin when Ranger is Kerberized

2020-03-14 Thread Hao Hao (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15414 to look at the new patch set (#4). Change subject: [java] fix Kudu Ranger plugin when Ranger is Kerberized

[kudu-CR] [subprocess] copy subprocess JAR to be next to the master binary

2020-03-14 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15438 ) Change subject: [subprocess] copy subprocess JAR to be next to the master binary .. Patch Set 1: I think we can remove this too :

[kudu-CR] KUDU-2780: create thread for auto-rebalancing

2020-03-14 Thread Hannah Nguyen (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14177 to look at the new patch set (#22). Change subject: KUDU-2780: create thread for auto-rebalancing

[kudu-CR] [subprocess] copy subprocess JAR to be next to the master binary

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15438 ) Change subject: [subprocess] copy subprocess JAR to be next to the master binary .. Patch Set 1: Verified+1 Unrelated flaky failure. --

[kudu-CR] [subprocess] copy subprocess JAR to be next to the master binary

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [subprocess] copy subprocess JAR to be next to the master binary .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/15438 To

[kudu-CR] KUDU-2780: create thread for auto-rebalancing

2020-03-14 Thread Hannah Nguyen (Code Review)
Hannah Nguyen has posted comments on this change. ( http://gerrit.cloudera.org:8080/14177 ) Change subject: KUDU-2780: create thread for auto-rebalancing .. Patch Set 20: (2 comments) rify

[kudu-CR] [build] address compilation warnings from GCC 8.3.1

2020-03-14 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15437 ) Change subject: [build] address compilation warnings from GCC 8.3.1 .. Patch Set 1: Verified+1 unrelated build fluke in IWYU:

[kudu-CR] [build] address compilation warnings from GCC 8.3.1

2020-03-14 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: [build] address compilation warnings from GCC 8.3.1 .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/15437 To

[kudu-CR] [ranger] parameterize the Java binary path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15415 ) Change subject: [ranger] parameterize the Java binary path for the subprocess .. Patch Set 4: (2 comments)

[kudu-CR] [ranger] parameterize the Java binary path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15415 to look at the new patch set (#4). Change subject: [ranger] parameterize the Java binary path for the subprocess

[kudu-CR] [subprocess] copy subprocess JAR to be next to the master binary

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15438 Change subject: [subprocess] copy subprocess JAR to be next to the master binary .. [subprocess] copy subprocess JAR to be next to the master

[kudu-CR] [build] address compilation warnings from GCC 8.3.1

2020-03-14 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15437 Change subject: [build] address compilation warnings from GCC 8.3.1 .. [build] address compilation warnings from GCC 8.3.1 While

[kudu-CR] [ranger] fix incorrect authz enforcement in Ranger authz provider

2020-03-14 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/15436 ) Change subject: [ranger] fix incorrect authz enforcement in Ranger authz provider .. Patch Set 1: (2 comments)

[kudu-CR] [ranger] validate the JAR file path for the subprocess

2020-03-14 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/15435 ) Change subject: [ranger] validate the JAR file path for the subprocess .. Patch Set 1: (1 comment)

[kudu-CR] [ranger] parameterize the Java binary path for the subprocess

2020-03-14 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/15415 ) Change subject: [ranger] parameterize the Java binary path for the subprocess .. Patch Set 3: (1 comment)

[kudu-CR] [java] fix Kudu Ranger plugin when Ranger is Kerberized

2020-03-14 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/15414 ) Change subject: [java] fix Kudu Ranger plugin when Ranger is Kerberized .. Patch Set 3: (1 comment)

[kudu-CR] [ranger] validate the JAR file path for the subprocess

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15435 ) Change subject: [ranger] validate the JAR file path for the subprocess .. Patch Set 1: -Code-Review Looking at some other build

[kudu-CR] [ranger] validate the JAR file path for the subprocess

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15435 ) Change subject: [ranger] validate the JAR file path for the subprocess .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [ranger] parameterize the Java binary path for the subprocess

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15415 ) Change subject: [ranger] parameterize the Java binary path for the subprocess .. Patch Set 3: Code-Review+1 (1 comment)

[kudu-CR] [java] fix Kudu Ranger plugin when Ranger is Kerberized

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15414 ) Change subject: [java] fix Kudu Ranger plugin when Ranger is Kerberized .. Patch Set 3: (1 comment)

[kudu-CR] [ranger] fix incorrect authz enforcement in Ranger authz provider

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15436 ) Change subject: [ranger] fix incorrect authz enforcement in Ranger authz provider .. Patch Set 1: (1 comment)

[kudu-CR] [ranger] pass 'principal' and 'keytab' to the subprocess

2020-03-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15416 ) Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess .. Patch Set 3: (4 comments)

[kudu-CR] [ranger] parameterize the Java binary path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15415 to look at the new patch set (#3). Change subject: [ranger] parameterize the Java binary path for the subprocess

[kudu-CR] [ranger] pass 'principal' and 'keytab' to the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15416 to look at the new patch set (#3). Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess

[kudu-CR] [ranger] validate the JAR file path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15435 Change subject: [ranger] validate the JAR file path for the subprocess .. [ranger] validate the JAR file path for the subprocess Change-Id:

[kudu-CR] [ranger] pass 'principal' and 'keytab' to the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15416 ) Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess .. Patch Set 3: (2 comments)

[kudu-CR] [ranger] parameterize the Java binary path for the subprocess

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15415 ) Change subject: [ranger] parameterize the Java binary path for the subprocess .. Patch Set 3: (5 comments)

[kudu-CR] [java] fix Kudu Ranger plugin when Ranger is Kerberized

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15414 ) Change subject: [java] fix Kudu Ranger plugin when Ranger is Kerberized .. Patch Set 3: (6 comments)

[kudu-CR] [java] fix Kudu Ranger plugin when Ranger is Kerberized

2020-03-14 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15414 to look at the new patch set (#3). Change subject: [java] fix Kudu Ranger plugin when Ranger is Kerberized

[kudu-CR] [ranger] fix incorrect authz enforcement in Ranger authz provider

2020-03-14 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15436 Change subject: [ranger] fix incorrect authz enforcement in Ranger authz provider .. [ranger] fix incorrect authz enforcement in Ranger

[kudu-CR] [java] use KuduTestHarness in TestNegotiation

2020-03-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15434 ) Change subject: [java] use KuduTestHarness in TestNegotiation .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [java] use KuduTestHarness in TestNegotiation

2020-03-14 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15434 ) Change subject: [java] use KuduTestHarness in TestNegotiation .. [java] use KuduTestHarness in TestNegotiation Change-Id:

[kudu-CR] [java] use KuduTestHarness in TestConnectionCache

2020-03-14 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15431 ) Change subject: [java] use KuduTestHarness in TestConnectionCache .. [java] use KuduTestHarness in TestConnectionCache

[kudu-CR] [java] use KuduTestHarness in TestConnectionCache

2020-03-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15431 ) Change subject: [java] use KuduTestHarness in TestConnectionCache .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [util] Changes to allow serializing/de-serializing BlockBloomFilter to a custom format

2020-03-14 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15396 ) Change subject: [util] Changes to allow serializing/de-serializing BlockBloomFilter to a custom format .. [util] Changes to

[kudu-CR] [java] use KuduTestHarness in TestNegotiation

2020-03-14 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15434 ) Change subject: [java] use KuduTestHarness in TestNegotiation .. Patch Set 2: Verified+1 unrelated tests failure in

[kudu-CR] [java] use KuduTestHarness in TestNegotiation

2020-03-14 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: [java] use KuduTestHarness in TestNegotiation .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/15434 To unsubscribe,

[kudu-CR] [java] use KuduTestHarness in TestNegotiation

2020-03-14 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15434 to look at the new patch set (#2). Change subject: [java] use KuduTestHarness in TestNegotiation ..