[kudu-CR] [sentry] add privilege scope validation to SentryAuthzProvider

2019-03-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12500 ) Change subject: [sentry] add privilege scope validation to SentryAuthzProvider .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2019-03-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 10: (9 comments) http://gerrit.cloudera.org:8080/#/c/

[kudu-CR] tserver: correct usage of col idx in SplitKeyRange

2019-03-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12707 ) Change subject: tserver: correct usage of col idx in SplitKeyRange .. Patch Set 1: This could use a test, but thought I'd put it on your

[kudu-CR] tserver: correct usage of col idx in SplitKeyRange

2019-03-08 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12707 Change subject: tserver: correct usage of col idx in SplitKeyRange .. tserver: correct usage of col idx in SplitKeyRange Change-Id: If444

[kudu-CR](branch-1.9.x) [known issues] the scalability of location/rack awareness

2019-03-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12706 ) Change subject: [known_issues] the scalability of location/rack awareness .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#

[kudu-CR] [java] Allow delete operations with extra columns set

2019-03-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12705 ) Change subject: [java] Allow delete operations with extra columns set .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/12

[kudu-CR](branch-1.9.x) [known issues] the scalability of location/rack awareness

2019-03-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12706 ) Change subject: [known_issues] the scalability of location/rack awareness .. Patch Set 3: (8 comments) http://gerrit.cloudera.org:8080/#

[kudu-CR] [java] Allow delete operations with extra columns set

2019-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12705 ) Change subject: [java] Allow delete operations with extra columns set .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c

[kudu-CR](branch-1.9.x) [known issues] the scalability of location/rack awareness

2019-03-08 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12706 ) Change subject: [known_issues] the scalability of location/rack awareness .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080

[kudu-CR](branch-1.9.x) [known issues] the scalability of location/rack awareness

2019-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12706 ) Change subject: [known_issues] the scalability of location/rack awareness .. Patch Set 3: (7 comments) http://gerrit.cloudera.org:808

[kudu-CR](branch-1.9.x) [known issues] the scalability of location/rack awareness

2019-03-08 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins, Andrew Wong, Adar Dembo, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12706 to look at the new patch set (#3). Change subject: [known_issues] the scalability of location/rack awareness ...

[kudu-CR] KUDU-2543 pt 3 java: pass around authz tokens

2019-03-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12279 ) Change subject: KUDU-2543 pt 3 java: pass around authz tokens .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:80

[kudu-CR](branch-1.9.x) [known issues] the scalability of location/rack awareness

2019-03-08 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12706 ) Change subject: [known_issues] the scalability of location/rack awareness .. Patch Set 2: (7 comments) http://gerrit.cloudera.org:808

[kudu-CR](branch-1.9.x) [known issues] the scalability of location/rack awareness

2019-03-08 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12706 to look at the new patch set (#2). Change subject: [known_issues] the scalability of location/rack awareness .. [kn

[kudu-CR](branch-1.9.x) [known issues] add info on location assignment performance

2019-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12706 Change subject: [known_issues] add info on location assignment performance .. [known_issues] add info on location assignment performance

[kudu-CR] [java] Allow delete operations with extra columns set

2019-03-08 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12705 Change subject: [java] Allow delete operations with extra columns set .. [java] Allow delete operations with extra columns set Before thi

[kudu-CR] KUDU-2543 pt 3 java: pass around authz tokens

2019-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12279 ) Change subject: KUDU-2543 pt 3 java: pass around authz tokens .. Patch Set 5: Code-Review+2 (1 comment) http://gerrit.cloudera.org:80

[kudu-CR] [gutil] fix build on macOS

2019-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12703 ) Change subject: [gutil] fix build on macOS .. [gutil] fix build on macOS The newly-added sysinfo-test consists of scenarios fo

[kudu-CR](gh-pages) update website for 1.9.0 release

2019-03-08 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12704 Change subject: update website for 1.9.0 release .. update website for 1.9.0 release Note: Since we removed the Quickstart VM, I've repla

[kudu-CR] [gutil] fix build on macOS

2019-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12703 Change subject: [gutil] fix build on macOS .. [gutil] fix build on macOS The newly-added sysinfo-test consists of scenarios for Linux-s

[kudu-CR] [gutil] fix build on macOS

2019-03-08 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12703 ) Change subject: [gutil] fix build on macOS .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12703 To u

[kudu-CR] binary jar: publish script should sign artifacts

2019-03-08 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12691 ) Change subject: binary jar: publish script should sign artifacts .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/12691/1

[kudu-CR] KUDU-2711 (part 2): use a RWMutex for TSDescriptor

2019-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12614 ) Change subject: KUDU-2711 (part 2): use a RWMutex for TSDescriptor .. Patch Set 2: Code-Review+1 (1 comment) http://gerrit.cloudera.o

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

2019-03-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12318 ) Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. Patch Set 7: Verified+1 Code-Review+1 -- To view, visit ht

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

2019-03-08 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/12318 To unsu

[kudu-CR] binary jar: publish script should sign artifacts

2019-03-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12691 ) Change subject: binary jar: publish script should sign artifacts .. Patch Set 1: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

2019-03-08 Thread Brian McDevitt (Code Review)
Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12318 ) Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. Patch Set 7: Looks like a flaky test: http://jenkins.kud

[kudu-CR] KUDU-2038-p2: support for index metadata management

2019-03-08 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12701 Change subject: KUDU-2038-p2: support for index metadata management .. KUDU-2038-p2: support for index metadata management The purpose of this