[kudu-CR] build: enable Java flaky test reporting

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12043 ) Change subject: build: enable Java flaky test reporting .. Patch Set 8: > > > Currently we retry all Java tests in dist_test.py. Should w

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

2019-04-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:808

[kudu-CR] client: remove a warning

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12909 ) Change subject: client: remove a warning .. client: remove a warning ../../src/kudu/client/schema.cc:624:27: warning: moving a

[kudu-CR] client: remove a warning

2019-04-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12909 ) Change subject: client: remove a warning .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12909 To uns

[kudu-CR] WIP [master] introduced SentryAuthzCache

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12833 ) Change subject: WIP [master] introduced SentryAuthzCache .. Patch Set 6: (7 comments) http://gerrit.cloudera.org:8080/#/c/12833/6/src/ku

[kudu-CR] remove schema copies from RowBlock and RowBuilder

2019-04-01 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12907 to look at the new patch set (#4). Change subject: remove schema copies from RowBlock and RowBuilder .

[kudu-CR] client: remove a warning

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12909 Change subject: client: remove a warning .. client: remove a warning ../../src/kudu/client/schema.cc:624:27: warning: moving a temporary

[kudu-CR] java: ensure KuduTestHarness or RetryRule in every test

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12872 ) Change subject: java: ensure KuduTestHarness or RetryRule in every test .. Patch Set 5: Verified+1 Overriding Jenkins, unrelated TSAN tes

[kudu-CR] java: ensure KuduTestHarness or RetryRule in every test

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/12872 ) Change subject: java: ensure KuduTestHarness or RetryRule in every test .. Removed reviewer Kudu Jenkins with the following votes:

[kudu-CR] WIP [master] introduced SentryAuthzCache

2019-04-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12833 ) Change subject: WIP [master] introduced SentryAuthzCache .. Patch Set 6: (28 comments) http://gerrit.cloudera.org:8080/#/c/12833/2//C

[kudu-CR] java: ensure KuduTestHarness or RetryRule in every test

2019-04-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12872 ) Change subject: java: ensure KuduTestHarness or RetryRule in every test .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cl

[kudu-CR] WIP [master] introduced SentryAuthzCache

2019-04-01 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12833 to look at the new patch set (#6). Change subject: WIP [master] introduced SentryAuthzCache

[kudu-CR] build: enable Java flaky test reporting

2019-04-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12043 ) Change subject: build: enable Java flaky test reporting .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/

[kudu-CR] java: add support for flaky test reporting

2019-04-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12042 ) Change subject: java: add support for flaky test reporting .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:80

[kudu-CR] java: add support for flaky test reporting

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new patch set (#8) to the change originally created by Mike Percy. ( http://gerrit.cloudera.org:8080/12042 ) Change subject: java: add support for flaky test reporting .. java: add support for flaky tes

[kudu-CR] java: add support for flaky test reporting

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12042 ) Change subject: java: add support for flaky test reporting .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/12042/7/java/

[kudu-CR] remove schema copies from RowBlock and RowBuilder

2019-04-01 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12907 to look at the new patch set (#3). Change subject: remove schema copies from RowBlock and RowBuilder .

[kudu-CR] remove schema copies from RowBlock and RowBuilder

2019-04-01 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12907 to look at the new patch set (#2). Change subject: remove schema copies from RowBlock and RowBuilder .

[kudu-CR] build: enable Java flaky test reporting

2019-04-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12043 ) Change subject: build: enable Java flaky test reporting .. Patch Set 7: Code-Review+2 > > Currently we retry all Java tests in dist_test.

[kudu-CR] remove schema copies from RowBlock and RowBuilder

2019-04-01 Thread Adar Dembo (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12907 to review the following change. Change subject: remove schema copies from RowBlock and RowBuilder .. remove schema cop

[kudu-CR] java: ensure KuduTestHarness or RetryRule in every test

2019-04-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12872 ) Change subject: java: ensure KuduTestHarness or RetryRule in every test .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cl

[kudu-CR] wip sentry: generate authz tokens

2019-04-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12897 ) Change subject: wip sentry: generate authz tokens .. Patch Set 1: (7 comments) left a few comments skimming through http://gerrit.cl

[kudu-CR] java: add support for flaky test reporting

2019-04-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12042 ) Change subject: java: add support for flaky test reporting .. Patch Set 7: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c

[kudu-CR] docs: remove extraneous space

2019-04-01 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12906 ) Change subject: docs: remove extraneous space .. Patch Set 1: Verified+1 Our friend: RemoteKsckTest.TestClusterWithLocation -- To vi

[kudu-CR] docs: remove extraneous space

2019-04-01 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: docs: remove extraneous space .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/12906 To unsubscribe, visit http://gerrit

[kudu-CR] docs: remove extraneous space

2019-04-01 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12906 ) Change subject: docs: remove extraneous space .. docs: remove extraneous space Change-Id: I0a2d4b6b8e5e09affc4c1a239e3a61d4c58a7

[kudu-CR] docs: remove extraneous space

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12906 ) Change subject: docs: remove extraneous space .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12906 To u

[kudu-CR] docs: remove extraneous space

2019-04-01 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12906 ) Change subject: docs: remove extraneous space .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12906 To

[kudu-CR] docs: remove extraneous space

2019-04-01 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12906 Change subject: docs: remove extraneous space .. docs: remove extraneous space Change-Id: I0a2d4b6b8e5e09affc4c1a239e3a61d4c58a72f8 --- M

[kudu-CR] [curl] Fix error status of timeout

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12898 ) Change subject: [curl] Fix error status of timeout .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12898

[kudu-CR] [curl] Fix error status of timeout

2019-04-01 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12898 ) Change subject: [curl] Fix error status of timeout .. [curl] Fix error status of timeout This commit fix the returned status of c

[kudu-CR] KUDU-2755: [Java] Upgrade protoc to 3.7.1

2019-04-01 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12904 ) Change subject: KUDU-2755: [Java] Upgrade protoc to 3.7.1 .. KUDU-2755: [Java] Upgrade protoc to 3.7.1 This fixes build failures

[kudu-CR] KUDU-2755: [Java] Upgrade protoc to 3.7.1

2019-04-01 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12904 ) Change subject: KUDU-2755: [Java] Upgrade protoc to 3.7.1 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:80

[kudu-CR] KUDU-2755: [Java] Upgrade protoc to 3.7.1

2019-04-01 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12904 Change subject: KUDU-2755: [Java] Upgrade protoc to 3.7.1 .. KUDU-2755: [Java] Upgrade protoc to 3.7.1 This fixes build failures due to a