[kudu-CR] [thirdparty] Bump RapidJSON version to 1.1.0

2019-03-04 Thread Yingchun Lai (Code Review)
Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/12643 ) Change subject: [thirdparty] Bump RapidJSON version to 1.1.0 .. Patch Set 6: (6 comments) > (6 comments) > > > looks like lots of

[kudu-CR] KUDU-2721: support ranges in CPU lists

2019-03-04 Thread Tim Armstrong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12657 to look at the new patch set (#6). Change subject: KUDU-2721: support ranges in CPU lists .. KUDU-2721:

[kudu-CR] [thirdparty] Bump RapidJSON version to 1.1.0

2019-03-04 Thread Yingchun Lai (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12643 to look at the new patch set (#6). Change subject: [thirdparty] Bump RapidJSON version to 1.1.0

[kudu-CR] tserver: remove DEPRECATED range predicates

2019-03-04 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12661 Change subject: tserver: remove DEPRECATED_range_predicates .. tserver: remove DEPRECATED_range_predicates This field hasn't been

[kudu-CR] KUDU-2721: support ranges in CPU lists

2019-03-04 Thread Tim Armstrong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12657 to look at the new patch set (#5). Change subject: KUDU-2721: support ranges in CPU lists .. KUDU-2721:

[kudu-CR] KUDU-2721: support ranges in CPU lists

2019-03-04 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12657 ) Change subject: KUDU-2721: support ranges in CPU lists .. Patch Set 5: Attempted to fix IWYU errors -- To view, visit

[kudu-CR] KUDU-2721: support ranges in CPU lists

2019-03-04 Thread Tim Armstrong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12657 to look at the new patch set (#4). Change subject: KUDU-2721: support ranges in CPU lists .. KUDU-2721:

[kudu-CR] KUDU-2721: support ranges in CPU lists

2019-03-04 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12657 ) Change subject: KUDU-2721: support ranges in CPU lists .. Patch Set 3: LMK if the hand-rolled parsing code is acceptable. I have

[kudu-CR] KUDU-2721: support ranges in CPU lists

2019-03-04 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/12657 ) Change subject: KUDU-2721: support ranges in CPU lists .. KUDU-2721: support ranges in CPU lists Generalise the logic to handle

[kudu-CR] [thirdparty] Bump RapidJSON version to 1.1.0

2019-03-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12643 ) Change subject: [thirdparty] Bump RapidJSON version to 1.1.0 .. Patch Set 5: (6 comments) > looks like lots of test failures from this

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

2019-03-04 Thread Adar Dembo (Code Review)
Adar Dembo 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+1 -- To view, visit

[kudu-CR] KUDU-2038-p1: Introduce CRoaring

2019-03-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12588 ) Change subject: KUDU-2038-p1: Introduce CRoaring .. Patch Set 2: (9 comments) http://gerrit.cloudera.org:8080/#/c/12588/2//COMMIT_MSG

[kudu-CR] [docker] Use only the kudu binary in the kudu image

2019-03-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12518 ) Change subject: [docker] Use only the kudu binary in the kudu image .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] [tools] Support running the master and tablet server via the kudu binary

2019-03-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12517 ) Change subject: [tools] Support running the master and tablet server via the kudu binary .. Patch Set 17: (1 comment)

[kudu-CR] WIP: KUDU-2724: Binary jar build on OSX should specify target macOS version

2019-03-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12655 ) Change subject: WIP: KUDU-2724: Binary jar build on OSX should specify target macOS version .. Patch Set 1: Code-Review+1 (1 comment)

[kudu-CR] WIP: KUDU-2724: Binary jar build on OSX should specify target macOS version

2019-03-04 Thread Mike Percy (Code Review)
Mike Percy has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12655 Change subject: WIP: KUDU-2724: Binary jar build on OSX should specify target macOS version .. WIP: KUDU-2724: Binary jar build on OSX

[kudu-CR] KUDU-2645: tablet: Support deduplication of deleted rows in MergeIterator

2019-03-04 Thread Mike Percy (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12205 to look at the new patch set (#8). Change subject: KUDU-2645: tablet: Support deduplication of deleted rows in MergeIterator

[kudu-CR] [tools] Support running the master and tablet server via the kudu binary

2019-03-04 Thread Grant Henke (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12517 to look at the new patch set (#17). Change subject: [tools] Support running the master and tablet server via the kudu

[kudu-CR] KUDU-2645: tablet: Support deduplication of deleted rows in MergeIterator

2019-03-04 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12205 to look at the new patch set (#7). Change subject: KUDU-2645: tablet: Support deduplication of deleted rows in MergeIterator

[kudu-CR] WIP: Expose RPC interface for diff scan

2019-03-04 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12592 to look at the new patch set (#2). Change subject: WIP: Expose RPC interface for diff scan .. WIP:

[kudu-CR] [tools] Support running the master and tablet server via the kudu binary

2019-03-04 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12517 ) Change subject: [tools] Support running the master and tablet server via the kudu binary .. Patch Set 15: (1 comment)

[kudu-CR] [tools] Support running the master and tablet server via the kudu binary

2019-03-04 Thread Grant Henke (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12517 to look at the new patch set (#16). Change subject: [tools] Support running the master and tablet server via the kudu

[kudu-CR] [master] introduce TTL-based cache

2019-03-04 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12652 ) Change subject: [master] introduce TTL-based cache .. Patch Set 1: (2 comments)

[kudu-CR] [master] introduce TTL-based cache

2019-03-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12652 ) Change subject: [master] introduce TTL-based cache .. Patch Set 1: (8 comments)

[kudu-CR] [docs] Fix table name in Impala docs

2019-03-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12534 ) Change subject: [docs] Fix table name in Impala docs .. Patch Set 3: (1 comment)

[kudu-CR] WIP [master] introduced SentryAuthzCache

2019-03-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12653 Change subject: WIP [master] introduced SentryAuthzCache .. WIP [master] introduced SentryAuthzCache This patch incorporate a

[kudu-CR] [master] introduce TTL-based cache

2019-03-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12652 Change subject: [master] introduce TTL-based cache .. [master] introduce TTL-based cache Change-Id:

[kudu-CR](branch-1.9.x) [docker] Fix build to work outside of Git WD

2019-03-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12650 ) Change subject: [docker] Fix build to work outside of Git WD .. Patch Set 1: Code-Review+2 Don't think I'm cutting an RC3, but wouldn't

[kudu-CR](branch-1.9.x) [docker] Fix build to work outside of Git WD

2019-03-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12650 ) Change subject: [docker] Fix build to work outside of Git WD .. Patch Set 1: -Code-Review Er, though maybe wait for 1.9.0 to release

[kudu-CR] KUDU-2670: Part 1: Build ScanToken by splitSizeBytes

2019-03-04 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by splitSizeBytes .. Patch Set 6: (9 comments) Thank you for the update Yao, it's