[kudu-CR] add a tool to create table

2019-10-11 Thread YangSong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14306 to look at the new patch set (#13). Change subject: add a tool to create table

[kudu-CR] add a tool to create table

2019-10-11 Thread YangSong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14306 to look at the new patch set (#12). Change subject: add a tool to create table

[kudu-CR] add a tool to create table

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14306 ) Change subject: add a tool to create table .. Patch Set 11: > I can't understand the failure in the jenkins. In `IWYU.log` no > error

[kudu-CR] KUDU-2800 Test long bootstrapping tablet replicas

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14300 ) Change subject: KUDU-2800 Test long bootstrapping tablet replicas .. Patch Set 15: (2 comments) http://gerrit.cloudera.org:8080/#/c/1

[kudu-CR] KUDU-2800 Test long bootstrapping tablet replicas

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14300 ) Change subject: KUDU-2800 Test long bootstrapping tablet replicas .. Patch Set 15: > > Build Failed > > > > http://jenkins.kudu.apac

[kudu-CR] KUDU-2800 Test long bootstrapping tablet replicas

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14300 ) Change subject: KUDU-2800 Test long bootstrapping tablet replicas .. Patch Set 15: > Build Failed > > http://jenkins.kudu.apache.org

[kudu-CR] KUDU-2800 Test long bootstrapping tablet replicas

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14300 ) Change subject: KUDU-2800 Test long bootstrapping tablet replicas .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/14

[kudu-CR] util: gscoped ptr -> unique ptr

2019-10-11 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14419 ) Change subject: util: gscoped_ptr -> unique_ptr .. util: gscoped_ptr -> unique_ptr This replaces all usages of gscoped_ptr with

[kudu-CR] util: gscoped ptr -> unique ptr

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14419 ) Change subject: util: gscoped_ptr -> unique_ptr .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/14419/1

[kudu-CR] util: gscoped ptr -> unique ptr

2019-10-11 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14419 ) Change subject: util: gscoped_ptr -> unique_ptr .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/14419/1//COMMIT_MSG Co

[kudu-CR] util: gscoped ptr -> unique ptr

2019-10-11 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Volodymyr Verovkin, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14419 to look at the new patch set (#2). Change subject: util: gscoped_ptr -> unique_ptr ..

[kudu-CR] util: gscoped ptr -> unique ptr

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14419 ) Change subject: util: gscoped_ptr -> unique_ptr .. Patch Set 1: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/14419/

[kudu-CR] util: gscoped ptr -> unique ptr

2019-10-11 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14419 Change subject: util: gscoped_ptr -> unique_ptr .. util: gscoped_ptr -> unique_ptr This replaces all usages of gscoped_ptr with unique_pt

[kudu-CR](branch-1.11.x) [docs] add 1.11.0 release notes

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14418 ) Change subject: [docs] add 1.11.0 release notes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/14418/1//COMMIT_MSG C

[kudu-CR] [docs] add 1.11.0 release notes

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14414 ) Change subject: [docs] add 1.11.0 release notes .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/14414/1/docs/release

[kudu-CR] [docs] add 1.11.0 release notes

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. ( http://gerrit.cloudera.org:8080/14414 ) Change subject: [docs] add 1.11.0 release notes .. Abandoned It seems I submitted the review against wrong version (master). I re-submitted t

[kudu-CR](branch-1.11.x) [docs] add 1.11.0 release notes

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14418 Change subject: [docs] add 1.11.0 release notes .. [docs] add 1.11.0 release notes Change-Id: I64ac48bfdcc4eb1b9c87af5be6dc9d8ecc219074

[kudu-CR] [docs] add 1.11.0 release notes

2019-10-11 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/14414 ) Change subject: [docs] add 1.11.0 release notes .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/14414/1/docs/release

[kudu-CR] KUDU-2800 Test long bootstrapping tablet replicas

2019-10-11 Thread Volodymyr Verovkin (Code Review)
Volodymyr Verovkin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14300 ) Change subject: KUDU-2800 Test long bootstrapping tablet replicas .. Patch Set 15: (16 comments) http://gerrit.cloudera.org:8080

[kudu-CR] KUDU-2800 Test long bootstrapping tablet replicas

2019-10-11 Thread Volodymyr Verovkin (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14300 to look at the new patch set (#15). Change subject: KUDU-2800 Test long bootstrapping tablet replicas

[kudu-CR] KUDU-2800 Test long bootstrapping tablet replicas

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14300 ) Change subject: KUDU-2800 Test long bootstrapping tablet replicas .. Patch Set 14: (16 comments) http://gerrit.cloudera.org:8080/#/c/

[kudu-CR] KUDU-2800 Test long bootstrapping tablet replicas

2019-10-11 Thread Volodymyr Verovkin (Code Review)
Volodymyr Verovkin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14300 ) Change subject: KUDU-2800 Test long bootstrapping tablet replicas .. Patch Set 14: (3 comments) http://gerrit.cloudera.org:8080/

[kudu-CR] KUDU-2800 Test long bootstrapping tablet replicas

2019-10-11 Thread Volodymyr Verovkin (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14300 to look at the new patch set (#14). Change subject: KUDU-2800 Test long bootstrapping tablet replicas

[kudu-CR](branch-1.11.x) [clock] log on CombineClocks() status only via VLOG

2019-10-11 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/14417 to review the following change. Change subject: [clock] log on CombineClocks() status only via VLOG ..

[kudu-CR] [clock] log on CombineClocks() status only via VLOG

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14416 ) Change subject: [clock] log on CombineClocks() status only via VLOG .. [clock] log on CombineClocks() status only via VLOG The

[kudu-CR] Update version to 1.12.0-SNAPSHOT

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14413 ) Change subject: Update version to 1.12.0-SNAPSHOT .. Update version to 1.12.0-SNAPSHOT Change-Id: Ifd7a8e1de46cc4ac548c5733732

[kudu-CR] Update version to 1.12.0-SNAPSHOT

2019-10-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14413 ) Change subject: Update version to 1.12.0-SNAPSHOT .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/14413

[kudu-CR] [clock] log on CombineClocks() status only via VLOG

2019-10-11 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14416 ) Change subject: [clock] log on CombineClocks() status only via VLOG .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloud

[kudu-CR] [clock] log on CombineClocks() status only via VLOG

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14416 Change subject: [clock] log on CombineClocks() status only via VLOG .. [clock] log on CombineClocks() status only via VLOG The logs abo

[kudu-CR] [build] Fix a bug for memory gc-itest

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14415 ) Change subject: [build] Fix a bug for memory_gc-itest .. Patch Set 1: Code-Review+2 Good catch! -- To view, visit http://gerrit.clou

[kudu-CR] [build] Fix a bug for memory gc-itest

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14415 ) Change subject: [build] Fix a bug for memory_gc-itest .. [build] Fix a bug for memory_gc-itest In CMakeList.txt, we should use

[kudu-CR] [docs] add 1.11.0 release notes

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14414 ) Change subject: [docs] add 1.11.0 release notes .. Patch Set 1: Verified+1 Unrelated failures in tests: * org.apache.kudu.mapreduce.

[kudu-CR] [docs] add 1.11.0 release notes

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: [docs] add 1.11.0 release notes .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/14414 To unsubscribe, visit http://ge

[kudu-CR] Update version to 1.12.0-SNAPSHOT

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14413 ) Change subject: Update version to 1.12.0-SNAPSHOT .. Patch Set 2: Verified+1 Unrelated failures of tests: * org.apache.kudu.flume.si

[kudu-CR] Update version to 1.12.0-SNAPSHOT

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed a vote on this change. Change subject: Update version to 1.12.0-SNAPSHOT .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/14413 To unsubscribe, visit http://

[kudu-CR] KUDU-2914: Rebalance tool support moving replicas from some specific tablet servers

2019-10-11 Thread Yifan Zhang (Code Review)
Yifan Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/14154 ) Change subject: KUDU-2914: Rebalance tool support moving replicas from some specific tablet servers .. Patch Set 6: > Patch Set 4: > >

[kudu-CR] add a tool to create table

2019-10-11 Thread YangSong (Code Review)
YangSong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14306 ) Change subject: add a tool to create table .. Patch Set 11: I can't understand the failure in the jenkins. In `IWYU.log` no error message

[kudu-CR] add a tool to create table

2019-10-11 Thread YangSong (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14306 to look at the new patch set (#11). Change subject: add a tool to create table .. add a tool

[kudu-CR] add a tool to create table

2019-10-11 Thread YangSong (Code Review)
YangSong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14306 ) Change subject: add a tool to create table .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/14306/10/src/kudu/tools/tool_a

[kudu-CR] add a tool to create table

2019-10-11 Thread YangSong (Code Review)
YangSong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14306 ) Change subject: add a tool to create table .. Patch Set 10: Yes, the test case TestAlterColumn failed because the string value of compress

[kudu-CR] [build] Fix a bug for memory gc-itest

2019-10-11 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14415 Change subject: [build] Fix a bug for memory_gc-itest .. [build] Fix a bug for memory_gc-itest In CMakeList.txt, we should use KUDU_TCMALLOC_A

[kudu-CR] add a tool to create table

2019-10-11 Thread YangSong (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14306 to look at the new patch set (#10). Change subject: add a tool to create table .. add a tool

[kudu-CR] add a tool to create table

2019-10-11 Thread YangSong (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14306 to look at the new patch set (#9). Change subject: add a tool to create table .. add a tool t

[kudu-CR] add a tool to create table

2019-10-11 Thread YangSong (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14306 to look at the new patch set (#8). Change subject: add a tool to create table .. add a tool t

[kudu-CR] Update version to 1.12.0-SNAPSHOT

2019-10-11 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/14413 to look at the new patch set (#2). Change subject: Update version to 1.12.0-SNAPSHOT .. Up

[kudu-CR] WIP [docs] Kudu 1.11 release notes draft

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. ( http://gerrit.cloudera.org:8080/14404 ) Change subject: WIP [docs] Kudu 1.11 release notes draft .. Abandoned Obsoleted by https://gerrit.cloudera.org/#/c/14414/ -- To view, visit htt

[kudu-CR] [docs] add 1.11.0 release notes

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14414 ) Change subject: [docs] add 1.11.0 release notes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/14414/1//COMMIT_MSG C

[kudu-CR] [docs] add 1.11.0 release notes

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14414 Change subject: [docs] add 1.11.0 release notes .. [docs] add 1.11.0 release notes Change-Id: I64ac48bfdcc4eb1b9c87af5be6dc9d8ecc219074

[kudu-CR] WIP [docs] Kudu 1.11 release notes draft

2019-10-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14404 ) Change subject: WIP [docs] Kudu 1.11 release notes draft .. Patch Set 7: (22 comments) http://gerrit.cloudera.org:8080/#/c/14404/7/do

[kudu-CR] WIP [docs] Kudu 1.11 release notes draft

2019-10-11 Thread Alexey Serbin (Code Review)
Hello honeyhexin, Attila Bukor, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, Grant Henke, Greg Solovyev, Hao Hao, Volodymyr Verovkin, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14404 to look at the new patch set (#8). Change subject: WIP