[kudu-CR] [catalog manager] correct the wrong comment

2019-09-16 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14233 ) Change subject: [catalog_manager] correct the wrong comment .. Patch Set 2: (1 comment)

[kudu-CR] [catalog manager] correct the wrong comment

2019-09-16 Thread honeyhexin (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14233 to look at the new patch set (#2). Change subject: [catalog_manager] correct the wrong comment

[kudu-CR] [catalog manager] correct the wrong comment

2019-09-16 Thread honeyhexin (Code Review)
honeyhexin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14241 Change subject: [catalog_manager] correct the wrong comment .. [catalog_manager] correct the wrong comment Change-Id:

[kudu-CR] [catalog manager] correct the wrong comment

2019-09-16 Thread honeyhexin (Code Review)
honeyhexin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14233 Change subject: [catalog_manager] correct the wrong comment .. [catalog_manager] correct the wrong comment Change-Id:

[kudu-CR] [test] Make the kudu-admin-test more robust by adding AssertEventually

2019-08-15 Thread honeyhexin (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14070 to look at the new patch set (#3). Change subject: [test] Make the kudu-admin-test more robust by adding AssertEventually

[kudu-CR] [test] Make the kudu-admin-test more robust by adding AssertEventually

2019-08-15 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14070 ) Change subject: [test] Make the kudu-admin-test more robust by adding AssertEventually .. Patch Set 2: (6 comments)

[kudu-CR] [test] Make the kudu-admin-test more robust by adding AssertEventually

2019-08-15 Thread honeyhexin (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14070 to look at the new patch set (#2). Change subject: [test] Make the kudu-admin-test more robust by adding AssertEventually

[kudu-CR] [test] Make the kudu-admin-test more rubust by adding AssertEventually

2019-08-15 Thread honeyhexin (Code Review)
honeyhexin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14070 Change subject: [test] Make the kudu-admin-test more rubust by adding AssertEventually .. [test] Make the kudu-admin-test more rubust by

[kudu-CR] KUDU-2915 Support to delete dead tservers from CLI

2019-08-12 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14048 ) Change subject: KUDU-2915 Support to delete dead tservers from CLI .. Patch Set 1: I agree that introducing `kudu cluster decommission`

[kudu-CR] KUDU-2915 Support to delete dead tservers from CLI

2019-08-12 Thread honeyhexin (Code Review)
honeyhexin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14048 Change subject: KUDU-2915 Support to delete dead tservers from CLI .. KUDU-2915 Support to delete dead tservers from CLI The patch

[kudu-CR] Remove redundant references of MasterServiceProxy

2019-08-12 Thread honeyhexin (Code Review)
honeyhexin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14047 Change subject: Remove redundant references of MasterServiceProxy .. Remove redundant references of MasterServiceProxy Change-Id:

[kudu-CR] [code style] Keep same code style in test files

2019-08-06 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14016 to look at the new patch set (#4). Change subject: [code style] Keep same code style in test files

[kudu-CR] [code style] Keep same code style in test files

2019-08-06 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14016 ) Change subject: [code style] Keep same code style in test files .. Patch Set 3: (6 comments)

[kudu-CR] [code style] Keep same code style in test files

2019-08-06 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14016 to look at the new patch set (#3). Change subject: [code style] Keep same code style in test files

[kudu-CR] [code style] Keep same code style in test files

2019-08-06 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14016 to look at the new patch set (#2). Change subject: [code style] Keep same code style in test files ..

[kudu-CR] [code style] Keep same code style in test files

2019-08-05 Thread honeyhexin (Code Review)
honeyhexin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14016 Change subject: [code style] Keep same code style in test files .. [code style] Keep same code style in test files Change all

[kudu-CR] tools: fix warning when compiling in ASAN and TSAN caused by ConvertToKuduPartialRow.

2019-07-31 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13974 ) Change subject: tools: fix warning when compiling in ASAN and TSAN caused by ConvertToKuduPartialRow. .. Patch Set 2: (2 comments)

[kudu-CR] tools: fix warning when compiling in ASAN and TSAN caused by ConvertToKuduPartialRow.

2019-07-31 Thread honeyhexin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13974 to look at the new patch set (#2). Change subject: tools: fix warning when compiling in ASAN and TSAN caused by ConvertToKuduPartialRow.

[kudu-CR] tools: fix warnning when compiling in ASAN and TSAN caused by ConvertToKuduPartialRow.

2019-07-31 Thread honeyhexin (Code Review)
honeyhexin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13974 Change subject: tools: fix warnning when compiling in ASAN and TSAN caused by ConvertToKuduPartialRow. .. tools: fix warnning when

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-29 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13833 ) Change subject: KUDU-2881 Support create/drop range partition by command line .. Patch Set 19: (2 comments)

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-29 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#19). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-29 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13833 ) Change subject: KUDU-2881 Support create/drop range partition by command line .. Patch Set 18: (1 comment)

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-29 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#18). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-29 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#17). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-29 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13833 ) Change subject: KUDU-2881 Support create/drop range partition by command line .. Patch Set 11: (6 comments)

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-29 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#16). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-28 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#15). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-28 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#14). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-28 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13833 ) Change subject: KUDU-2881 Support create/drop range partition by command line .. Patch Set 13: (14 comments)

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-28 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#13). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-24 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#12). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-24 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#11). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-24 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#10). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-24 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#9). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-24 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13833 ) Change subject: KUDU-2881 Support create/drop range partition by command line .. Patch Set 8: (11 comments)

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-24 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#8). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-22 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13833 ) Change subject: KUDU-2881 Support create/drop range partition by command line .. Patch Set 7: If the range partition now is unbounded,

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-22 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#7). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-21 Thread honeyhexin (Code Review)
honeyhexin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13833 ) Change subject: KUDU-2881 Support create/drop range partition by command line .. Patch Set 6: (18 comments)

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-21 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#6). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-21 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#5). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-17 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#4). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-17 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#3). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-17 Thread honeyhexin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Yao Xu, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13833 to look at the new patch set (#2). Change subject: KUDU-2881 Support create/drop range partition by command line

[kudu-CR] KUDU-2881 Support create/drop range partition by command line

2019-07-10 Thread honeyhexin (Code Review)
honeyhexin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13833 Change subject: KUDU-2881 Support create/drop range partition by command line .. KUDU-2881 Support create/drop range partition by command