[kudu-CR] KUDU-2437 Generate ScanToken from small chunks in tablet

2018-05-18 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/10406 ) Change subject: KUDU-2437 Generate ScanToken from small chunks in tablet .. Patch Set 4: Thank Todd for the question pointed out, we will

[kudu-CR] KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key"

2018-05-21 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10463 Change subject: KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key" .. KUDU-2447 Fix TS crashed with "NONE predicate

[kudu-CR] Merge branch 'master' of ssh://gerrit.cloudera.org:29418/kudu into gerrit2

2018-05-22 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10475 Change subject: Merge branch 'master' of ssh://gerrit.cloudera.org:29418/kudu into gerrit2 .. Merge branch 'master' of

[kudu-CR] Merge branch 'master' of ssh://gerrit.cloudera.org:29418/kudu into gerrit2

2018-05-22 Thread Yao Xu (Code Review)
Yao Xu has abandoned this change. ( http://gerrit.cloudera.org:8080/10475 ) Change subject: Merge branch 'master' of ssh://gerrit.cloudera.org:29418/kudu into gerrit2 .. Abandoned -- To view, visit

[kudu-CR] KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key"

2018-05-21 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10463 to look at the new patch set (#2). Change subject: KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key"

[kudu-CR] KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key"

2018-05-21 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/10463 ) Change subject: KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key" .. Patch Set 1: Ok, thanks Todd, I will fix this

[kudu-CR] KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key"

2018-05-22 Thread Yao Xu (Code Review)
Yao Xu has abandoned this change. ( http://gerrit.cloudera.org:8080/10463 ) Change subject: KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key" .. Abandoned -- To view, visit

[kudu-CR] KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key"

2018-05-25 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10514 Change subject: KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key" .. KUDU-2447 Fix TS crashed with "NONE predicate

[kudu-CR] KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key"

2018-05-25 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10514 to look at the new patch set (#2). Change subject: KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key"

[kudu-CR] KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key"

2018-05-31 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/10514 ) Change subject: KUDU-2447 Fix TS crashed with "NONE predicate can not be pushed into key" .. Patch Set 3: > (1 comment) > > Great find,

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-20 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#10). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-19 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#7). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-20 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#11). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-20 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#13). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-20 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#12). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-19 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#5). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-19 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#8). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-19 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#9). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-07-13 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/10406 ) Change subject: KUDU-2437 Split a tablet into primary key ranges by size .. Patch Set 21: (8 comments)

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-07-13 Thread Yao Xu (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#20). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-07-09 Thread Yao Xu (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#15). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-07-09 Thread Yao Xu (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#18). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-07-09 Thread Yao Xu (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#17). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-07-09 Thread Yao Xu (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#16). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-07-09 Thread Yao Xu (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#19). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-07-09 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/10406 ) Change subject: KUDU-2437 Split a tablet into primary key ranges by size .. Patch Set 19: (21 comments) Thanks for review, I already

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-20 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#14). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-09-16 Thread Yao Xu (Code Review)
Hello Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#22). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-09-16 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/10406 ) Change subject: KUDU-2437 Split a tablet into primary key ranges by size .. Patch Set 22: > (8 comments) > > I've got a high-level

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-09-16 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/10406 ) Change subject: KUDU-2437 Split a tablet into primary key ranges by size .. Patch Set 22: (10 comments) > (2 comments) > > This is

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-09-17 Thread Yao Xu (Code Review)
Hello Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#23). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-09-17 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/10406 ) Change subject: KUDU-2437 Split a tablet into primary key ranges by size .. Patch Set 23: (2 comments) Hi, Adar Dembo Thanks for review. I

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

2019-03-23 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#8) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

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

2019-03-23 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#10) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

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

2019-03-23 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. Patch Set 10: (22 comments) Hi, guys. I refactored part of the code for

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

2019-03-23 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#7) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by splitSizeBytes .. KUDU-2670: Part 1: Build

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

2019-03-23 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#9) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

[kudu-CR] KUDU-2514 Support extra config for table.

2019-02-28 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12468 ) Change subject: KUDU-2514 Support extra config for table. .. Patch Set 2: (1 comment) Hi, todd. Thanks for review.

[kudu-CR] KUDU-2670: split more scanner and add concurrent

2019-02-28 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: split more scanner and add concurrent .. Patch Set 2: Hi, grant. I will continue this work. -- To view, visit

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

2019-03-01 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#4) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by splitSizeBytes .. KUDU-2670: Part 1: Build

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

2019-03-01 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#5) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by splitSizeBytes .. KUDU-2670: Part 1: Build

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

2019-03-01 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by splitSizeBytes .. Patch Set 5: (9 comments) Hi, grant. I update this patch. I broke

[kudu-CR] KUDU-2514 Support extra config for table.

2019-03-05 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12468 ) Change subject: KUDU-2514 Support extra config for table. .. Patch Set 2: (2 comments) Hi, adar. Thanks for review.

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

2019-03-05 Thread Yao Xu (Code Review)
Yao Xu 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: Hi, grant and adar. Thanks for review. I will update

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

2019-03-01 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#3) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by splitSizeBytes .. KUDU-2670: Part 1: Build

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

2019-03-01 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#6) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by splitSizeBytes .. KUDU-2670: Part 1: Build

[kudu-CR] tserver: correct usage of col idx in SplitKeyRange

2019-03-10 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12707 ) Change subject: tserver: correct usage of col idx in SplitKeyRange .. Patch Set 3: Code-Review+1 -- To view, visit

[kudu-CR] KUDU-2514 Support extra config for table.

2019-02-13 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12468 Change subject: KUDU-2514 Support extra config for table. .. KUDU-2514 Support extra config for table. This patch implements the extra-config

[kudu-CR] KUDU-2514 Support extra config for table.

2019-02-13 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12468 to look at the new patch set (#2). Change subject: KUDU-2514 Support extra config for table. .. KUDU-2514 Support

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-05-16 Thread Yao Xu (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/12468 to look at the new patch set (#7). Change subject: KUDU-2514 Part 1: Support extra config for table.

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

2019-05-12 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#20) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

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

2019-05-12 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#19) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

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

2019-05-12 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#18) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

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

2019-05-12 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. Patch Set 20: (12 comments) Apologize for updating the patch late. I have

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

2019-05-16 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#28) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

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

2019-05-16 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#25) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

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

2019-05-16 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. Patch Set 26: (6 comments) Thanks for review.

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

2019-05-16 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#27) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

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

2019-05-17 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. Patch Set 28: I found that DefaultSourceTest.testScanWithKeyRange has a

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

2019-05-17 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#29) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

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

2019-05-17 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#30) to the change originally created by yangz. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. KUDU-2670: Part 1: Build ScanToken

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-05-17 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12468 ) Change subject: KUDU-2514 Part 1: Support extra config for table. .. Patch Set 8: (2 comments) Thanks adar. I will implement the C++ API

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

2019-05-17 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: Part 1: Build ScanToken by KeyRange .. Patch Set 30: Sorry, the problem has not been reproduced in local. The

[kudu-CR] KUDU-2802: C++ tableExists API optimizations

2019-05-27 Thread Yao Xu (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/13443 to look at the new patch set (#3). Change subject: KUDU-2802: C++ tableExists API optimizations

[kudu-CR] [client] C++ tableExists API optimizations

2019-05-27 Thread Yao Xu (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/13443 to look at the new patch set (#2). Change subject: [client] C++ tableExists API optimizations

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-04 Thread Yao Xu (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/12468 to look at the new patch set (#14). Change subject: KUDU-2514 Part 1: Support extra config for table.

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-04 Thread Yao Xu (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/12468 to look at the new patch set (#16). Change subject: KUDU-2514 Part 1: Support extra config for table.

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-04 Thread Yao Xu (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/12468 to look at the new patch set (#15). Change subject: KUDU-2514 Part 1: Support extra config for table.

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-04 Thread Yao Xu (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/12468 to look at the new patch set (#13). Change subject: KUDU-2514 Part 1: Support extra config for table.

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-04 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12468 ) Change subject: KUDU-2514 Part 1: Support extra config for table. .. Patch Set 16: (15 comments) Thanks for the comments. I made some

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-05 Thread Yao Xu (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/12468 to look at the new patch set (#17). Change subject: KUDU-2514 Part 1: Support extra config for table.

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-05 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12468 ) Change subject: KUDU-2514 Part 1: Support extra config for table. .. Patch Set 18: (4 comments)

[kudu-CR] KUDU-2670 Set --enable rowset compaction�lse flag for test of SplitKeyRange

2019-06-05 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13517 Change subject: KUDU-2670 Set --enable_rowset_compaction=false flag for test of SplitKeyRange .. KUDU-2670 Set

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-07 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12468 ) Change subject: KUDU-2514 Part 1: Support extra config for table. .. Patch Set 20: (4 comments)

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-07 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12468 to look at the new patch set (#20). Change subject: KUDU-2514 Part 1: Support extra config for table.

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-04 Thread Yao Xu (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/12468 to look at the new patch set (#12). Change subject: KUDU-2514 Part 1: Support extra config for table.

[kudu-CR] [test] Fix bug of TestSplitKeyRange

2019-06-06 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13530 ) Change subject: [test] Fix bug of TestSplitKeyRange .. Patch Set 1: The root cause: If created KeyRanges that covers all the DRSs in the

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-05 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12468 to look at the new patch set (#19). Change subject: KUDU-2514 Part 1: Support extra config for table.

[kudu-CR] KUDU-2670 Disable rowset compact for test of SplitKeyRange

2019-06-05 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13517 to look at the new patch set (#2). Change subject: KUDU-2670 Disable rowset compact for test of SplitKeyRange

[kudu-CR] Add seek before mode for CBTree to accelerate CheckRowDeleted in dms.

2019-06-05 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13516 ) Change subject: Add seek before mode for CBTree to accelerate CheckRowDeleted in dms. .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2514 Part 1: Support extra config for table.

2019-06-05 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12468 ) Change subject: KUDU-2514 Part 1: Support extra config for table. .. Patch Set 19: (1 comment) Fix conflict.

[kudu-CR] [test] Fix bug of TestSplitKeyRange

2019-06-06 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13530 Change subject: [test] Fix bug of TestSplitKeyRange .. [test] Fix bug of TestSplitKeyRange In theory, the data size of KeyRange is likely to

[kudu-CR] [test] Fix bug of TestSplitKeyRange

2019-06-06 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13530 ) Change subject: [test] Fix bug of TestSplitKeyRange .. Patch Set 1: If the tablet does not have DRS, the data size of result will be zero.

[kudu-CR] [client] C++ tableExists API optimizations

2019-05-27 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13443 Change subject: [client] C++ tableExists API optimizations .. [client] C++ tableExists API optimizations The C++ client `tableExists` should

[kudu-CR] [tools] Add get/set extra-configs for CLI tools

2019-06-14 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13649 Change subject: [tools] Add get/set extra-configs for CLI tools .. [tools] Add get/set extra-configs for CLI tools Change-Id:

[kudu-CR] [tools] Add get/set extra-configs for CLI tools

2019-06-14 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13649 to look at the new patch set (#2). Change subject: [tools] Add get/set extra-configs for CLI tools .. [tools] Add

[kudu-CR] KUDU-2823 Place tablet replicas based on range

2019-06-13 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13632 Change subject: KUDU-2823 Place tablet replicas based on range .. KUDU-2823 Place tablet replicas based on range When we add a new range to

[kudu-CR] KUDU-2823 Place tablet replicas based on range

2019-06-13 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#2). Change subject: KUDU-2823 Place tablet replicas based on range ..

[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-11 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13573 ) Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table .. Patch Set 5: (2 comments) Emm, the

[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-11 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13573 to look at the new patch set (#5). Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table

[kudu-CR] [test] Remove unused variables

2019-06-14 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13658 Change subject: [test] Remove unused variables .. [test] Remove unused variables Change-Id: I5b96dd87c2f189659f6a0e89c6fd2a54f00933fa --- M

[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-10 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13573 to look at the new patch set (#4). Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table

[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-11 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13573 ) Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table .. Patch Set 4: (2 comments)

[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-10 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13573 to look at the new patch set (#3). Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table

[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-10 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13573 Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table .. KUDU-2514 Part 2: Supports setting

[kudu-CR] KUDU-2823 Place tablet replicas based on range

2019-06-17 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on range .. Patch Set 7: (2 comments) Thanks Alexey, Todd for comments.

[kudu-CR] [tools] Add get/set extra-configs for CLI tools

2019-06-19 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13649 to look at the new patch set (#3). Change subject: [tools] Add get/set extra-configs for CLI tools ..

[kudu-CR] [docs] Add docs for extra configuration properties

2019-06-24 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13704 to look at the new patch set (#3). Change subject: [docs] Add docs for extra configuration properties

[kudu-CR] [docs] Add docs for extra configuration properties

2019-06-24 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13704 ) Change subject: [docs] Add docs for extra configuration properties .. Patch Set 3: (3 comments) Thanks for comments, it looks much better

[kudu-CR] [docs] Add docs for extra configuration properties

2019-06-24 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13704 to look at the new patch set (#4). Change subject: [docs] Add docs for extra configuration properties

  1   2   3   >