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

2020-03-16 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 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/15

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

2020-03-16 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 (#6). 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-16 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 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 th

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

2020-03-16 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 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/15

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

2020-03-16 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 4: (1 comment) http://gerrit.cloudera.org:808

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

2020-03-16 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 4: (1 comment) > Patch Set 4: Code-Review+1 >

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

2020-03-16 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 4: Unrelated flaky tests. -- To view, visit

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

2020-03-16 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 4: Verified+1 -- To view, visit http://gerrit

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

2020-03-16 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 u

[kudu-CR] [hms] disable direct SQL in MiniHms

2020-03-16 Thread Hao Hao (Code Review)
Hao Hao has abandoned this change. ( http://gerrit.cloudera.org:8080/11198 ) Change subject: [hms] disable direct SQL in MiniHms .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/11198 To unsubscribe, visit http://g

[kudu-CR] [spark] add 'local-cluster' mode for unit test

2020-03-16 Thread Hao Hao (Code Review)
Hao Hao has abandoned this change. ( http://gerrit.cloudera.org:8080/8585 ) Change subject: [spark] add 'local-cluster' mode for unit test .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/8585 To unsubscribe, visit

[kudu-CR] [spark] improve log message to warn users to pass keytab/principal

2020-03-16 Thread Hao Hao (Code Review)
Hao Hao has abandoned this change. ( http://gerrit.cloudera.org:8080/9730 ) Change subject: [spark] improve log message to warn users to pass keytab/principal .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/9730

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

2020-03-16 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 (#5). Change subject: [java] fix Kudu Ranger plugin when Ranger is Kerberized .

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

2020-03-16 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) http://gerrit.cloudera.org:8080/#/c/15

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

2020-03-16 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: (2 comments) http://gerrit.cloudera.org:80

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

2020-03-16 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 (#4). Change subject: [subprocess] copy subprocess JAR to be next to the master binary

[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) http://gerrit.cloudera.org:8080/#/c/15

[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) http://gerrit.cloudera.org:

[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) http://gerrit.cloudera.org:8080/#/c/15

[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) http://gerrit.cloudera.org:808

[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 u

[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 -- T

[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 unsu

[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 th

[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] 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) http://gerrit.cloudera.org:8080/#/c/1

[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)
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] 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) http://gerrit.cloudera.org:

[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) http://gerrit.cloudera.org:8080/#/c/1543

[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) http://gerrit.cloudera.org:8080/#/c/15

[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 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. -- To

[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 u

[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) http://gerrit.cloudera.org:8080/

[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] [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: I

[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) http://gerrit.cloudera.org:8080/#/c/1

[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) http://gerrit.cloudera.org:8080/

[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) http://gerrit.cloudera.org:8080/#/c/15

[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 authz

[kudu-CR] [java] fix Kudu Ranger plugin in a Kerberized env

2020-03-12 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 in a Kerberized env .. Patch Set 2: Verified+1 Unrelated flaky test -- To view, visit http:/

[kudu-CR] [java] fix Kudu Ranger plugin in a Kerberized env

2020-03-12 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [java] fix Kudu Ranger plugin in a Kerberized env .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/15414 To unsubscribe, vis

[kudu-CR] [ranger] fix the java binary path for the subprocess

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

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

2020-03-11 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 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/1

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

2020-03-11 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 (#2). Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess ...

[kudu-CR] [java] fix Kudu Ranger plugin in a Kerberized env

2020-03-11 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 in a Kerberized env .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/15414/1/

[kudu-CR] [java] fix Kudu Ranger plugin in a Kerberized env

2020-03-11 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/15414 to look at the new patch set (#2). Change subject: [java] fix Kudu Ranger plugin in a Kerberized env ..

[kudu-CR] [ranger] fix the java binary path for the subprocess

2020-03-11 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/15415 to look at the new patch set (#2). Change subject: [ranger] fix the java binary path for the subprocess ...

[kudu-CR] [java] fix Kudu Ranger plugin in a Kerberized env

2020-03-11 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15414 Change subject: [java] fix Kudu Ranger plugin in a Kerberized env .. [java] fix Kudu Ranger plugin in a Kerberized env Kudu Ranger plugin cur

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

2020-03-11 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15416 Change subject: [ranger] pass 'principal' and 'keytab' to the subprocess .. [ranger] pass 'principal' and 'keytab' to the subprocess This pat

[kudu-CR] [ranger] fix the java binary path for the subprocess

2020-03-11 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15415 Change subject: [ranger] fix the java binary path for the subprocess .. [ranger] fix the java binary path for the subprocess Change-Id: Ic8f2

[kudu-CR] [java] Fix checkstyle and spotbugs issues

2020-03-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15410 ) Change subject: [java] Fix checkstyle and spotbugs issues .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/1

[kudu-CR] KUDU-2972 Add Ranger authorization provider

2020-03-05 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15207 ) Change subject: KUDU-2972 Add Ranger authorization provider .. KUDU-2972 Add Ranger authorization provider Adds a new authz_provider

[kudu-CR] KUDU-2972 Add Ranger authorization provider

2020-03-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15207 ) Change subject: KUDU-2972 Add Ranger authorization provider .. Patch Set 29: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:808

[kudu-CR] subprocess: add server-side metrics

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15367 ) Change subject: subprocess: add server-side metrics .. Patch Set 3: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/15367/3/s

[kudu-CR] subprocess: report metrics even when calls fail

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15366 ) Change subject: subprocess: report metrics even when calls fail .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] subprocess: fix TestEchoSubprocess

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15362 ) Change subject: subprocess: fix TestEchoSubprocess .. subprocess: fix TestEchoSubprocess There were a number of issues with TestEcho

[kudu-CR] subprocess: fix TestEchoSubprocess

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15362 ) Change subject: subprocess: fix TestEchoSubprocess .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15362 To

[kudu-CR] [WIP] Ranger authorization provider

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15207 ) Change subject: [WIP] Ranger authorization provider .. Patch Set 26: (1 comment) http://gerrit.cloudera.org:8080/#/c/15207/26/src/kudu/mast

[kudu-CR] KUDU-2972 Add Ranger client

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15206 ) Change subject: KUDU-2972 Add Ranger client .. Patch Set 27: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15206 To unsub

[kudu-CR] KUDU-2972 Add Ranger client

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15206 ) Change subject: KUDU-2972 Add Ranger client .. Patch Set 26: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15206 To unsub

[kudu-CR] [java] KUDU-2972: Add Kudu Ranger plugin

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15074 ) Change subject: [java] KUDU-2972: Add Kudu Ranger plugin .. [java] KUDU-2972: Add Kudu Ranger plugin This commit adds the Kudu Range

[kudu-CR] [wip] subprocess: fix TestEchoSubprocess

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15362 ) Change subject: [wip] subprocess: fix TestEchoSubprocess .. Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/1536

[kudu-CR] [java] KUDU-2972: Add Kudu Ranger plugin

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [java] KUDU-2972: Add Kudu Ranger plugin .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/15074 To unsubscribe, visit http:/

[kudu-CR] [java] KUDU-2972: Add Kudu Ranger plugin

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15074 ) Change subject: [java] KUDU-2972: Add Kudu Ranger plugin .. Patch Set 20: Verified+1 Unrelated flaky test. -- To view, visit http://gerrit

[kudu-CR] [java] KUDU-2972: Add Kudu Ranger plugin

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15074 ) Change subject: [java] KUDU-2972: Add Kudu Ranger plugin .. Patch Set 19: (3 comments) http://gerrit.cloudera.org:8080/#/c/15074/19/java/ku

[kudu-CR] [java] KUDU-2972: Add Kudu Ranger plugin

2020-03-04 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/15074 to look at the new patch set (#20). Change subject: [java] KUDU-2972: Add Kudu Ranger plugin ...

[kudu-CR] [build] Use Gradle to build the hms-plugin jar

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15360 ) Change subject: [build] Use Gradle to build the hms-plugin jar .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8

[kudu-CR] [java] KUDU-2972: Add Kudu Ranger plugin

2020-03-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15074 ) Change subject: [java] KUDU-2972: Add Kudu Ranger plugin .. Patch Set 18: (4 comments) http://gerrit.cloudera.org:8080/#/c/15074/13/java/ku

[kudu-CR] [java] KUDU-2972: Add Kudu Ranger plugin

2020-03-04 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/15074 to look at the new patch set (#18). Change subject: [java] KUDU-2972: Add Kudu Ranger plugin ...

[kudu-CR] subprocess: plumb Java metrics into C++

2020-03-03 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15344 ) Change subject: subprocess: plumb Java metrics into C++ .. Patch Set 5: Code-Review+2 > Patch Set 4: Code-Review+1 > > +1 in case others wan

[kudu-CR] [java] KUDU-2972: add Kudu Ranger plugin

2020-03-03 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [java] KUDU-2972: add Kudu Ranger plugin .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/15074 To unsubscribe, visit http:/

[kudu-CR] subprocess: plumb Java metrics into C++

2020-03-03 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15344 ) Change subject: subprocess: plumb Java metrics into C++ .. Patch Set 4: Code-Review+1 +1 in case others want to take a look. -- To view, v

[kudu-CR] subprocess: plumb Java metrics into C++

2020-03-03 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15344 ) Change subject: subprocess: plumb Java metrics into C++ .. Patch Set 3: Code-Review+1 (2 comments) Thanks for the patch! Just some nits. An

[kudu-CR] [wip] KUDU-2971 p3: concurrent requests support for subprocess

2020-03-03 Thread Hao Hao (Code Review)
Hao Hao has abandoned this change. ( http://gerrit.cloudera.org:8080/14625 ) Change subject: [wip] KUDU-2971 p3: concurrent requests support for subprocess .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/14625 To

[kudu-CR] [hms] Fix testAlterTableHandler in later Hive versions

2020-03-03 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15349 ) Change subject: [hms] Fix testAlterTableHandler in later Hive versions .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloude

[kudu-CR] [java] KUDU-2972: add Kudu Ranger plugin

2020-03-03 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15074 ) Change subject: [java] KUDU-2972: add Kudu Ranger plugin .. Patch Set 12: (2 comments) http://gerrit.cloudera.org:8080/#/c/15074/12/java/ku

[kudu-CR] [java] KUDU-2972: add Kudu Ranger plugin

2020-03-03 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/15074 to look at the new patch set (#13). Change subject: [java] KUDU-2972: add Kudu Ranger plugin ...

[kudu-CR] KUDU-2972 Add Ranger client

2020-03-03 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15206 ) Change subject: KUDU-2972 Add Ranger client .. Patch Set 19: Code-Review+1 (5 comments) Overall looks good to me, thanks a lot for working

[kudu-CR] [hms] Fix testAlterTableHandler in later Hive versions

2020-03-03 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15349 ) Change subject: [hms] Fix testAlterTableHandler in later Hive versions .. Patch Set 1: Verified+1 Code-Review+2 -- To view, visit http://ge

[kudu-CR] [java] KUDU-2972: add Kudu Ranger plugin

2020-03-02 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/15074 to look at the new patch set (#12). Change subject: [java] KUDU-2972: add Kudu Ranger plugin

[kudu-CR] [java] subprocess: add a metrics message

2020-03-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15315 ) Change subject: [java] subprocess: add a metrics message .. Patch Set 2: Code-Review+1 (2 comments) Overall looks good to me, just a few ni

[kudu-CR] KUDU-2972 Add Ranger client

2020-03-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15206 ) Change subject: KUDU-2972 Add Ranger client .. Patch Set 16: (13 comments) http://gerrit.cloudera.org:8080/#/c/15206/16/src/kudu/ranger/ran

[kudu-CR] KUDU-2972 Add Ranger client

2020-03-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15206 ) Change subject: KUDU-2972 Add Ranger client .. Patch Set 16: Also it seems there is a IWYU failure for anger_client-test.cc. -- To view, v

[kudu-CR] [java] subprocess: add a metrics message

2020-03-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15315 ) Change subject: [java] subprocess: add a metrics message .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/15315/1/java/kudu

[kudu-CR] [java] KUDU-2972: add Kudu Ranger plugin

2020-03-01 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/15074 to look at the new patch set (#11). Change subject: [java] KUDU-2972: add Kudu Ranger plugin

[kudu-CR] [java] KUDU-2972: add Kudu Ranger plugin

2020-03-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15074 ) Change subject: [java] KUDU-2972: add Kudu Ranger plugin .. Patch Set 11: (10 comments) http://gerrit.cloudera.org:8080/#/c/15074/9/java/ku

[kudu-CR] KUDU-2972 Add Ranger client

2020-02-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15206 ) Change subject: KUDU-2972 Add Ranger client .. Patch Set 10: (12 comments) The failure test is probably due to the recent change I had in

<    1   2   3   4   5   6   7   8   9   10   >