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

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15433 ) Change subject: [java] use KuduTestHarness in TestSecurity .. Patch Set 1: Looks a bit odd: TestSecurity failed only under TSAN

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

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( 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 TestSecurity

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15433 Change subject: [java] use KuduTestHarness in TestSecurity .. [java] use KuduTestHarness in TestSecurity Change-Id:

[kudu-CR] [client] Add C++ API to accept BlockBloomFilter directly

2020-03-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15424 ) Change subject: [client] Add C++ API to accept BlockBloomFilter directly .. Patch Set 2: (1 comment)

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

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15431 Change subject: [java] use KuduTestHarness in TestConnectionCache .. [java] use KuduTestHarness in TestConnectionCache Change-Id:

[kudu-CR] [client] Add C++ API to accept BlockBloomFilter directly

2020-03-13 Thread Bankim Bhavsar (Code Review)
Hello Thomas Tauber-Marshall, Kudu Jenkins, Adar Dembo, Wenzhe Zhou, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15424 to look at the new patch set (#2). Change subject: [client] Add C++ API to accept BlockBloomFilter directly

[kudu-CR] [client] Add C++ API to accept BlockBloomFilter directly

2020-03-13 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15424 ) Change subject: [client] Add C++ API to accept BlockBloomFilter directly .. Patch Set 1: (2 comments)

[kudu-CR] [docker] Add an Impala quickstart image

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15393 ) Change subject: [docker] Add an Impala quickstart image .. [docker] Add an Impala quickstart image This patch adds an `impala`

[kudu-CR] [docker] Add an Impala quickstart image

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15393 ) Change subject: [docker] Add an Impala quickstart image .. Patch Set 7: Code-Review+2 -- To view, visit

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

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15423 ) Change subject: [java] use KuduTestHarness in TestConnectToCluster .. [java] use KuduTestHarness in TestConnectToCluster

[kudu-CR] [quickstart] Add an Apache Impala quickstart guide

2020-03-13 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15397 ) Change subject: [quickstart] Add an Apache Impala quickstart guide .. Patch Set 7: Code-Review+2 I ran through the Impala quickstart.

[kudu-CR] [java] Upgrade to Gradle 6.2.2

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15430 ) Change subject: [java] Upgrade to Gradle 6.2.2 .. [java] Upgrade to Gradle 6.2.2 This patch makes the major version upgrade to

[kudu-CR] [docker] Add an Impala quickstart image

2020-03-13 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15393 ) Change subject: [docker] Add an Impala quickstart image .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] [docker] Add an Impala quickstart image

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15393 ) Change subject: [docker] Add an Impala quickstart image .. Patch Set 6: (1 comment)

[kudu-CR] [docker] Add an Impala quickstart image

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15393 ) Change subject: [docker] Add an Impala quickstart image .. Patch Set 7: Verified+1 -- To view, visit

[kudu-CR] [docker] Add an Impala quickstart image

2020-03-13 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15393 to look at the new patch set (#7). Change subject: [docker] Add an Impala quickstart image

[kudu-CR] [docker] Add an Impala quickstart image

2020-03-13 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15393 ) Change subject: [docker] Add an Impala quickstart image .. Patch Set 6: Code-Review+1 (1 comment)

[kudu-CR] [java] Upgrade to Gradle 6.2.2

2020-03-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15430 ) Change subject: [java] Upgrade to Gradle 6.2.2 .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15430 To

[kudu-CR] [docker] Add an Impala quickstart image

2020-03-13 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15393 to look at the new patch set (#6). Change subject: [docker] Add an Impala quickstart image

[kudu-CR] [quickstart] Add an Apache Impala quickstart guide

2020-03-13 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, Andrew Wong, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15397 to look at the new patch set (#7). Change subject: [quickstart] Add an Apache Impala quickstart guide

[kudu-CR] [docker] Add an Impala quickstart image

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15393 ) Change subject: [docker] Add an Impala quickstart image .. Patch Set 6: (2 comments)

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

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15423 ) Change subject: [java] use KuduTestHarness in TestConnectToCluster .. Patch Set 4: Code-Review+2 -- To view, visit

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

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15423 ) Change subject: [java] use KuduTestHarness in TestConnectToCluster .. Patch Set 3: (1 comment)

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

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

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

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [java] use KuduTestHarness in TestConnectToCluster .. Removed Code-Review+2 by Grant Henke -- To view, visit http://gerrit.cloudera.org:8080/15423 To unsubscribe,

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

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15423 ) Change subject: [java] use KuduTestHarness in TestConnectToCluster .. Patch Set 3: (1 comment)

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

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15423 ) Change subject: [java] use KuduTestHarness in TestConnectToCluster .. Patch Set 3: Code-Review+2 -- To view, visit

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

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

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

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15423 ) Change subject: [java] use KuduTestHarness in TestConnectToCluster .. Patch Set 2: (1 comment)

[kudu-CR] [java] Upgrade to Gradle 6.2.2

2020-03-13 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15430 to look at the new patch set (#3). Change subject: [java] Upgrade to Gradle 6.2.2 ..

[kudu-CR] [java] Upgrade to Gradle 6.2.2

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15430 ) Change subject: [java] Upgrade to Gradle 6.2.2 .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/15430/1/java/gradlew

[kudu-CR] [java] Upgrade to Gradle 6.2.2

2020-03-13 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15430 to look at the new patch set (#2). Change subject: [java] Upgrade to Gradle 6.2.2 ..

[kudu-CR] [java] Upgrade to Gradle 6.2.2

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15430 ) Change subject: [java] Upgrade to Gradle 6.2.2 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/15430/1/java/gradlew

[kudu-CR] [java] Upgrade to Gradle 6.2.2

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15430 ) Change subject: [java] Upgrade to Gradle 6.2.2 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/15430/1/java/gradlew

[kudu-CR] [java] Upgrade to Gradle 6.2.2

2020-03-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15430 ) Change subject: [java] Upgrade to Gradle 6.2.2 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/15430/1/java/gradlew

[kudu-CR] [gradle] remove failOnNoMatchingTests

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15428 ) Change subject: [gradle] remove failOnNoMatchingTests .. [gradle] remove failOnNoMatchingTests I found it confusing when

[kudu-CR] [tests] use 768-bit RSA keys for crypto in tests

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15426 ) Change subject: [tests] use 768-bit RSA keys for crypto in tests .. [tests] use 768-bit RSA keys for crypto in tests With

[kudu-CR] [java] Upgrade to Gradle 6.2.2

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15430 Change subject: [java] Upgrade to Gradle 6.2.2 .. [java] Upgrade to Gradle 6.2.2 This patch makes the major version upgrade to Gradle

[kudu-CR] [backup] fix log info

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15429 ) Change subject: [backup] fix log info .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/15429 To

[kudu-CR] [backup] fix log info

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15429 ) Change subject: [backup] fix log info .. [backup] fix log info Change-Id: I4eae75374c21253acfcbf410617b6b39cfbed7bf

[kudu-CR] [tests] use 768-bit RSA keys for crypto in tests

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15426 ) Change subject: [tests] use 768-bit RSA keys for crypto in tests .. Patch Set 1: Code-Review+2 -- To view, visit

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

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15423 ) Change subject: [java] use KuduTestHarness in TestConnectToCluster .. Patch Set 2: (1 comment)

[kudu-CR] [gradle] remove failOnNoMatchingTests

2020-03-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15428 ) Change subject: [gradle] remove failOnNoMatchingTests .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [backup] fix log info

2020-03-13 Thread Yifan Zhang (Code Review)
Yifan Zhang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15429 Change subject: [backup] fix log info .. [backup] fix log info Change-Id: I4eae75374c21253acfcbf410617b6b39cfbed7bf --- M

[kudu-CR] [gradle] remove failOnNoMatchingTests

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: [gradle] remove failOnNoMatchingTests .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/15428 To unsubscribe, visit

[kudu-CR] [gradle] remove failOnNoMatchingTests

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15428 ) Change subject: [gradle] remove failOnNoMatchingTests .. Patch Set 1: Verified+1 unrelated dist-test snafu: failed to download task

[kudu-CR] [gradle] remove failOnNoMatchingTests

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15428 Change subject: [gradle] remove failOnNoMatchingTests .. [gradle] remove failOnNoMatchingTests I found it confusing when gradle

[kudu-CR] KUDU-3063: Set a ratio as multiplication factor for memkind malloc usable size

2020-03-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15313 ) Change subject: KUDU-3063: Set a ratio as multiplication factor for memkind_malloc_usable_size .. KUDU-3063: Set a ratio as

[kudu-CR] wip KUDU-2975 p1: introduce WAL dir manager

2020-03-13 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15427 to look at the new patch set (#2). Change subject: wip KUDU-2975 p1: introduce WAL dir manager .. wip KUDU-2975

[kudu-CR] wip KUDU-2975 p1: introduce WAL dir manager

2020-03-13 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15427 Change subject: wip KUDU-2975 p1: introduce WAL dir manager .. wip KUDU-2975 p1: introduce WAL dir manager DONT_BUILD While the WAL dir