[kudu-CR] SelectionVector: pad extra bits with zeroes in constructor

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13009 ) Change subject: SelectionVector: pad extra bits with zeroes in constructor .. SelectionVector: pad extra bits with zeroes in

[kudu-CR] WIP [docs] updated master-related scenarios w.r.t. CM

2019-04-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13022 ) Change subject: WIP [docs] updated master-related scenarios w.r.t. CM .. Patch Set 1: (1 comment)

[kudu-CR] generic iterators: refactor MergeIterator for whole-block-copy optimization

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13010 ) Change subject: generic_iterators: refactor MergeIterator for whole-block-copy optimization .. Removed reviewer Kudu Jenkins

[kudu-CR] generic iterators: refactor MergeIterator for whole-block-copy optimization

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13010 ) Change subject: generic_iterators: refactor MergeIterator for whole-block-copy optimization .. Patch Set 5: Verified+1 Overriding

[kudu-CR] generic iterators: refactor MergeIterator for whole-block-copy optimization

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13010 ) Change subject: generic_iterators: refactor MergeIterator for whole-block-copy optimization .. Patch Set 5: > As far as I can tell you

[kudu-CR] [master] remove tserver service proto from libmaster

2019-04-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13054 ) Change subject: [master] remove tserver_service_proto from libmaster .. [master] remove tserver_service_proto from libmaster

[kudu-CR] generic iterators: MergeIterator whole block copy optimization

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13011 ) Change subject: generic_iterators: MergeIterator whole block copy optimization .. Patch Set 6: Verified+1 Overriding Jenkins, unrelated

[kudu-CR] generic iterators: MergeIterator whole block copy optimization

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13011 ) Change subject: generic_iterators: MergeIterator whole block copy optimization .. generic_iterators: MergeIterator whole block

[kudu-CR] generic iterators: MergeIterator whole block copy optimization

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13011 ) Change subject: generic_iterators: MergeIterator whole block copy optimization .. Removed reviewer Kudu Jenkins with the

[kudu-CR] Bump Sentry version

2019-04-17 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13059 Change subject: Bump Sentry version .. Bump Sentry version The motivation is to get access to SENTRY-2471, which includes a fix for syncing

[kudu-CR] [docs] updated docs w.r.t. collocation practices

2019-04-17 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12997 ) Change subject: [docs] updated docs w.r.t. collocation practices .. Patch Set 4: (1 comment)

[kudu-CR] generic iterators: refactor MergeIterator for whole-block-copy optimization

2019-04-17 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13010 ) Change subject: generic_iterators: refactor MergeIterator for whole-block-copy optimization .. Patch Set 5: Code-Review+2 Thanks for

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

2019-04-17 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 10: (8 comments) Just a few small comments. I think

[kudu-CR] [docs] updated docs w.r.t. collocation practices

2019-04-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12997 ) Change subject: [docs] updated docs w.r.t. collocation practices .. Patch Set 4: (1 comment)

[kudu-CR] generic iterators: refactor MergeIterator for whole-block-copy optimization

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13010 ) Change subject: generic_iterators: refactor MergeIterator for whole-block-copy optimization .. generic_iterators: refactor

[kudu-CR] rowblock: add copying functionality

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13008 ) Change subject: rowblock: add copying functionality .. rowblock: add copying functionality This patch adds RowBlock::CopyTo, a

[kudu-CR] generic iterators: MergeIterator whole block copy optimization

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13011 ) Change subject: generic_iterators: MergeIterator whole block copy optimization .. Patch Set 6: (3 comments)

[kudu-CR] generic iterators: MergeIterator whole block copy optimization

2019-04-17 Thread Adar Dembo (Code Review)
Hello Mike Percy, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13011 to look at the new patch set (#6). Change subject: generic_iterators: MergeIterator whole block copy optimization

[kudu-CR] generic iterators: MergeIterator whole block copy optimization

2019-04-17 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13011 ) Change subject: generic_iterators: MergeIterator whole block copy optimization .. Patch Set 5: Code-Review+1 (3 comments) Nice work

[kudu-CR] SelectionVector: pad extra bits with zeroes in constructor

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13009 ) Change subject: SelectionVector: pad extra bits with zeroes in constructor .. Patch Set 5: (1 comment)

[kudu-CR] rowblock: add copying functionality

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13008 ) Change subject: rowblock: add copying functionality .. Patch Set 5: Verified+1 Overriding Jenkins, known Java test flakes. -- To view,

[kudu-CR] rowblock: add copying functionality

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13008 ) Change subject: rowblock: add copying functionality .. Removed reviewer Kudu Jenkins with the following votes: * Verified-1 by

[kudu-CR] Bump Sentry version

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13059 ) Change subject: Bump Sentry version .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/13059/1//COMMIT_MSG Commit

[kudu-CR] [docs] updated docs w.r.t. collocation practices

2019-04-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12997 ) Change subject: [docs] updated docs w.r.t. collocation practices .. Patch Set 4: (1 comment) > Patch Set 4: > > (1 comment)

[kudu-CR] generic iterators: MergeIterator whole block copy optimization

2019-04-17 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13011 ) Change subject: generic_iterators: MergeIterator whole block copy optimization .. Patch Set 6: Code-Review+2 (1 comment)

[kudu-CR] [ksck] Make ksck tool more concurrent

2019-04-17 Thread Yingchun Lai (Code Review)
Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/13024 ) Change subject: [ksck] Make ksck tool more concurrent .. Patch Set 3: (4 comments)

[kudu-CR] [ksck] Make ksck tool more concurrent

2019-04-17 Thread Yingchun Lai (Code Review)
Hello Will Berkeley, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13024 to look at the new patch set (#4). Change subject: [ksck] Make ksck tool more concurrent

[kudu-CR] [catalog manager] simplify lifecycle of authz provider

2019-04-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13062 Change subject: [catalog_manager] simplify lifecycle of authz provider .. [catalog_manager] simplify lifecycle of authz provider This

[kudu-CR] WIP [master] RPC knobs to control authz privileges cache

2019-04-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13063 Change subject: WIP [master] RPC knobs to control authz privileges cache .. WIP [master] RPC knobs to control authz privileges cache

[kudu-CR] [ksck] Make ksck tool more concurrent

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13024 ) Change subject: [ksck] Make ksck tool more concurrent .. Patch Set 4: (2 comments) The test failures look legitimate.

[kudu-CR] [catalog manager] simplify lifecycle of authz provider

2019-04-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13062 ) Change subject: [catalog_manager] simplify lifecycle of authz provider .. Patch Set 1: Verified+1 unrelated flake in

[kudu-CR] [catalog manager] simplify lifecycle of authz provider

2019-04-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: [catalog_manager] simplify lifecycle of authz provider .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13062 To

[kudu-CR] [ksck] Make ksck tool more concurrent

2019-04-17 Thread Yingchun Lai (Code Review)
Hello Will Berkeley, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13024 to look at the new patch set (#3). Change subject: [ksck] Make ksck tool more concurrent

[kudu-CR] [ksck] Make ksck tool more concurrent

2019-04-17 Thread Yingchun Lai (Code Review)
Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/13024 ) Change subject: [ksck] Make ksck tool more concurrent .. Patch Set 2: (21 comments)

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

2019-04-17 Thread helifu (Code Review)
Hello Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12588 to look at the new patch set (#3). Change subject: KUDU-2038-p1: Introduce CRoaring .. KUDU-2038-p1:

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

2019-04-17 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12588 ) Change subject: KUDU-2038-p1: Introduce CRoaring .. Patch Set 2: (6 comments) Sorry for the late reply.

[kudu-CR] WIP [docs] updated master-related scenarios w.r.t. CM

2019-04-17 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/13022 ) Change subject: WIP [docs] updated master-related scenarios w.r.t. CM .. Patch Set 1: (1 comment)

[kudu-CR] [master] remove tserver service proto from libmaster

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13054 ) Change subject: [master] remove tserver_service_proto from libmaster .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [ksck] Make ksck tool more concurrent

2019-04-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13024 ) Change subject: [ksck] Make ksck tool more concurrent .. Patch Set 3: (4 comments)