[kudu-CR] [clock] cleanup on accessing HybridClock::next timestamp

2020-02-12 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, Volodymyr Verovkin, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15216 to look at the new patch set (#2). Change subject: [clock] cleanup on accessing HybridClock::next_timestamp_

[kudu-CR] [clock] cleanup on accessing HybridClock::next timestamp

2020-02-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15216 Change subject: [clock] cleanup on accessing HybridClock::next_timestamp_ .. [clock] cleanup on accessing HybridClock::next_timestamp_

[kudu-CR] [clock] update on Clock interface

2020-02-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15215 Change subject: [clock] update on Clock interface .. [clock] update on Clock interface This patch re-factors Clock-related classes:

[kudu-CR] [WIP] Ranger client

2020-02-12 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15206 ) Change subject: [WIP] Ranger client .. Patch Set 3: (3 comments) Only looked at the protobuf definition and ranger client as it is a wip.

[kudu-CR] [hybrid clock] KUDU-3048 introduce new clock metrics

2020-02-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15212 ) Change subject: [hybrid clock] KUDU-3048 introduce new clock metrics .. Patch Set 3: > Build Failed > >

[kudu-CR] [hybrid clock] KUDU-3048 introduce new clock metrics

2020-02-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15212 ) Change subject: [hybrid clock] KUDU-3048 introduce new clock metrics .. Patch Set 2: (2 comments)

[kudu-CR] [hybrid clock] KUDU-3048 introduce new clock metrics

2020-02-12 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15212 to look at the new patch set (#3). Change subject: [hybrid clock] KUDU-3048 introduce new clock metrics

[kudu-CR] [hybrid clock] KUDU-3048 introduce new clock metrics

2020-02-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15212 ) Change subject: [hybrid clock] KUDU-3048 introduce new clock metrics .. Patch Set 2: (2 comments)

[kudu-CR] [hybrid clock] KUDU-3048 introduce new clock metrics

2020-02-12 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15212 to look at the new patch set (#2). Change subject: [hybrid clock] KUDU-3048 introduce new clock metrics

[kudu-CR] [hybrid clock] KUDU-3048 introduce new clock metrics

2020-02-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15212 ) Change subject: [hybrid clock] KUDU-3048 introduce new clock metrics .. Patch Set 1: (2 comments)

[kudu-CR] [client] KUDU-2483 Add IN Bloom filter predicate to C++ client

2020-02-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15122 ) Change subject: [client] KUDU-2483 Add IN Bloom filter predicate to C++ client .. Patch Set 8: (1 comment)

[kudu-CR] [client] KUDU-2483 Add IN Bloom filter predicate to C++ client

2020-02-12 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15122 ) Change subject: [client] KUDU-2483 Add IN Bloom filter predicate to C++ client .. Patch Set 8: (1 comment)

[kudu-CR] [WIP] Ranger authorization provider

2020-02-12 Thread Attila Bukor (Code Review)
Hello Tidy Bot, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15207 to look at the new patch set (#3). Change subject: [WIP] Ranger authorization provider .. [WIP]

[kudu-CR] [client] KUDU-2483 Add IN Bloom filter predicate to C++ client

2020-02-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15122 ) Change subject: [client] KUDU-2483 Add IN Bloom filter predicate to C++ client .. Patch Set 8: Code-Review+2 (1 comment)

[kudu-CR] [WIP] Ranger client

2020-02-12 Thread Attila Bukor (Code Review)
Hello Tidy Bot, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15206 to look at the new patch set (#3). Change subject: [WIP] Ranger client .. [WIP] Ranger client

[kudu-CR] [client] KUDU-2483 Add IN Bloom filter predicate to C++ client

2020-02-12 Thread Bankim Bhavsar (Code Review)
Hello Kudu Jenkins, helifu, Yao Xu, Adar Dembo, ZhangYao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15122 to look at the new patch set (#8). Change subject: [client] KUDU-2483 Add IN Bloom filter predicate to C++ client

[kudu-CR] [hybrid clock] KUDU-3048 introduce new clock metrics

2020-02-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15212 ) Change subject: [hybrid clock] KUDU-3048 introduce new clock metrics .. Patch Set 1: (3 comments)

[kudu-CR] [hybrid clock] KUDU-3048 introduce new clock metrics

2020-02-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15212 Change subject: [hybrid clock] KUDU-3048 introduce new clock metrics .. [hybrid clock] KUDU-3048 introduce new clock metrics

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

2020-02-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15074 ) Change subject: [java] KUDU-2972: add Kudu Ranger plugin .. Patch Set 4: (17 comments) How do you feel about predicating this change on

[kudu-CR] [clock] introduce time source auto selection

2020-02-12 Thread Volodymyr Verovkin (Code Review)
Volodymyr Verovkin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15161 ) Change subject: [clock] introduce time source auto selection .. Patch Set 8: (1 comment)

[kudu-CR] [cpp] KUDU-2971: protobuf-based wrapper for subprocesses

2020-02-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15185 ) Change subject: [cpp] KUDU-2971: protobuf-based wrapper for subprocesses .. Patch Set 1: (24 comments) May need to modify the dist-test

[kudu-CR] subprocess: add an optional sleep to Echo service

2020-02-12 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15184 ) Change subject: subprocess: add an optional sleep to Echo service .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] subprocess: add an optional sleep to Echo service

2020-02-12 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15184 ) Change subject: subprocess: add an optional sleep to Echo service .. subprocess: add an optional sleep to Echo service I found this

[kudu-CR] KUDU-2483 Integrate BlockBloomFilter with ColumnPredicate on server side

2020-02-12 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15034 ) Change subject: KUDU-2483 Integrate BlockBloomFilter with ColumnPredicate on server side .. Patch Set 13: > Patch Set 13:

[kudu-CR] [client] KUDU-2483 Add IN Bloom filter predicate to C++ client

2020-02-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15122 ) Change subject: [client] KUDU-2483 Add IN Bloom filter predicate to C++ client .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] [client] KUDU-2483 Add IN Bloom filter predicate to C++ client

2020-02-12 Thread Bankim Bhavsar (Code Review)
Hello Kudu Jenkins, helifu, Yao Xu, Adar Dembo, ZhangYao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15122 to look at the new patch set (#7). Change subject: [client] KUDU-2483 Add IN Bloom filter predicate to C++ client

[kudu-CR] [client] KUDU-2483 Add IN Bloom filter predicate to C++ client

2020-02-12 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15122 ) Change subject: [client] KUDU-2483 Add IN Bloom filter predicate to C++ client .. Patch Set 6: (3 comments)

[kudu-CR] [java] KUDU-2971: add an outbound queue

2020-02-12 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15210 ) Change subject: [java] KUDU-2971: add an outbound queue .. Patch Set 2: Verified+1 Unrelated flaky test. -- To view, visit

[kudu-CR] [java] KUDU-2971: add an outbound queue

2020-02-12 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [java] KUDU-2971: add an outbound queue .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/15210 To unsubscribe, visit

[kudu-CR] [java] KUDU-2971: add an outbound queue

2020-02-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15210 ) Change subject: [java] KUDU-2971: add an outbound queue .. Patch Set 2: (8 comments)

[kudu-CR] [WIP] Ranger client

2020-02-12 Thread Attila Bukor (Code Review)
Hello Tidy Bot, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15206 to look at the new patch set (#2). Change subject: [WIP] Ranger client .. [WIP] Ranger client

[kudu-CR] [clock] introduce time source auto selection

2020-02-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15161 ) Change subject: [clock] introduce time source auto selection .. Patch Set 8: (1 comment)

[kudu-CR] [WIP] Ranger authorization provider

2020-02-12 Thread Attila Bukor (Code Review)
Hello Tidy Bot, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15207 to look at the new patch set (#2). Change subject: [WIP] Ranger authorization provider .. [WIP]