[kudu-CR] remove outdated information from consensus.txt

2019-04-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13023 ) Change subject: remove outdated information from consensus.txt .. Patch Set 1: I wonder if we should remove this doc entirely. There is

[kudu-CR] WIP [master] add RPC to reset AuthzProvider's cache

2019-04-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: WIP [master] add RPC to reset AuthzProvider's cache .. Patch Set 2: (7 comments) All nits pretty much, but unsure what

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 3: Code-Review+1 (4 comments)

[kudu-CR] hms: skip drop table notification log for external table

2019-04-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13073 ) Change subject: hms: skip drop table notification log for external table .. Patch Set 1: (6 comments)

[kudu-CR] [authz] new SentryAuthzProvider's caching strategy

2019-04-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13069 ) Change subject: [authz] new SentryAuthzProvider's caching strategy .. Patch Set 4: (12 comments) Looks good for the most part; I've

[kudu-CR] [authz] new SentryAuthzProvider's caching strategy

2019-04-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13069 ) Change subject: [authz] new SentryAuthzProvider's caching strategy .. Patch Set 4: (1 comment)

[kudu-CR] master: use AuthzProvider to generate authz tokens

2019-04-22 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13013 ) Change subject: master: use AuthzProvider to generate authz tokens .. Patch Set 9: Code-Review+2 (2 comments)

[kudu-CR] master: use AuthzProvider to generate authz tokens

2019-04-22 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13013 ) Change subject: master: use AuthzProvider to generate authz tokens .. master: use AuthzProvider to generate authz tokens This

[kudu-CR] master: use AuthzProvider to generate authz tokens

2019-04-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13013 ) Change subject: master: use AuthzProvider to generate authz tokens .. Patch Set 9: (2 comments)

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 4: Code-Review+1 -- To view, visit

[kudu-CR](branch-1.9.x) known issues: document fault tolerant scan memory issue

2019-04-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13081 ) Change subject: known_issues: document fault tolerant scan memory issue .. Patch Set 1: Verified+1 Overriding Jenkins, unrelated test

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 4: (2 comments)

[kudu-CR] [backup] Fix fromMs override option

2019-04-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13080 ) Change subject: [backup] Fix fromMs override option .. Patch Set 1: (3 comments)

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-22 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 3: (4 comments)

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-22 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12877 to look at the new patch set (#4). Change subject: [sentry] enable sentry integration for master stress test

[kudu-CR](branch-1.9.x) known issues: document fault tolerant scan memory issue

2019-04-22 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13081 ) Change subject: known_issues: document fault tolerant scan memory issue .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] [backup] Fix fromMs override option

2019-04-22 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13080 Change subject: [backup] Fix fromMs override option .. [backup] Fix fromMs override option Fix the logic for deciding when to lookup old

[kudu-CR](branch-1.9.x) known issues: document fault tolerant scan memory issue

2019-04-22 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13081 Change subject: known_issues: document fault tolerant scan memory issue .. known_issues: document fault tolerant scan memory issue

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 3: Code-Review+1 (1 comment)

[kudu-CR] dist-test: support RUN FLAKY ONLY

2019-04-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13072 ) Change subject: dist-test: support RUN_FLAKY_ONLY .. Patch Set 1: Verified+1 Overriding Jenkins, unrelated TSAN flaky C++ test failure.

[kudu-CR] dist-test: support RUN FLAKY ONLY

2019-04-22 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13072 ) Change subject: dist-test: support RUN_FLAKY_ONLY .. Removed reviewer Kudu Jenkins with the following votes: * Verified-1 by

[kudu-CR](gh-pages) Add authorization blog

2019-04-22 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13075 ) Change subject: Add authorization blog .. Patch Set 1: A rendered version of this posts can be seen here:

[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-04-22 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Greg Solovyev, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12119 to look at the new patch set (#11). Change subject: [blog] a blogpost about location awareness in Kudu

[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-04-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12119 ) Change subject: [blog] a blogpost about location awareness in Kudu .. Patch Set 11: (9 comments)

[kudu-CR](gh-pages) Add authorization blog

2019-04-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13075 ) Change subject: Add authorization blog .. Patch Set 1: Code-Review+2 Looks good! Tested all the links and gave it another read. -- To

[kudu-CR](gh-pages) Add authorization blog

2019-04-22 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13075 ) Change subject: Add authorization blog .. Add authorization blog Change-Id: Ib52db53d74211efcc49e9e122e133e812066966b

[kudu-CR](gh-pages) Add authorization blog

2019-04-22 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13075 ) Change subject: Add authorization blog .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/13075 To

[kudu-CR](gh-pages) Add authorization blog

2019-04-22 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13075 Change subject: Add authorization blog .. Add authorization blog Change-Id: Ib52db53d74211efcc49e9e122e133e812066966b --- A

[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-04-22 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12119 ) Change subject: [blog] a blogpost about location awareness in Kudu .. Patch Set 11: Code-Review+2 -- To view, visit