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

2018-05-17 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 mod

[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 can

[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-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] 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 ssh://gerrit.cloudera.o

[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 http://gerrit.cloudera.org:80

[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 http://gerrit.cloudera.org:808

[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 can

[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, th

[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 (#6). 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-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-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-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 (#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-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 (#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-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 (#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 (#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 (#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 (#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 updated

[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-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 (#21). 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) http://gerrit.cloudera.org:8080/#/c/1

[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: (10 comments) > (2 comments) > > This is looking

[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 question

[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 h

[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 e

[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. http://gerrit.cloudera.o

[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 ht

[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 ScanT

[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 ScanT

[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 ScanT

[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 th

[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 ScanT

[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. http://gerrit.cloudera.

[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 th

[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 http://gerrit.cloudera.or

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

2019-03-22 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 ScanT

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

2019-03-22 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 b

[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 b

[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 th

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

2019-05-10 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#11) 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-11 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#12) 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-11 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#13) 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-11 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#14) 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-11 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#15) 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-11 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#16) 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-11 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#17) 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 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 (#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 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-14 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#21) 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-14 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#22) 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-14 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#23) 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-14 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 23: (18 comments) http://gerrit.cloudera.org:8080/#/c/12323/20/j

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

2019-05-14 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#24) 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-14 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 24: (1 comment) Add tablet not null check in KeyRanage's constru

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

2019-05-16 Thread Yao Xu (Code Review)
Hello 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 (#3). Change subject: KUDU-2514 Part1: Support extra config for table. ...

[kudu-CR] KUDU-2514 Part1: 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 (#4). Change subject: KUDU-2514 Part1: Support extra config for table. .

[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 (#5). Change subject: KUDU-2514 Part 1: Support extra config for table.

[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 (#6). Change subject: KUDU-2514 Part 1: Support extra config for table.

[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-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. http://gerrit.cloudera.org:

[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-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-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 pro

[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-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 erro

[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 and

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

2019-05-21 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13388 Change subject: KUDU-2514 Part 1: Support extra config for table. .. KUDU-2514 Part 1: Support extra config for table. We have thousands of ta

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

2019-05-21 Thread Yao Xu (Code Review)
Yao Xu has abandoned this change. ( http://gerrit.cloudera.org:8080/13388 ) Change subject: KUDU-2514 Part 1: Support extra config for table. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/13388 To unsubscribe, v

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

2019-05-21 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 (#9). Change subject: KUDU-2514 Part 1: Support extra config for table.

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

2019-05-21 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 (#10). Change subject: KUDU-2514 Part 1: Support extra config for table. ...

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

2019-05-21 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 10: Well, it seems that the failure of jenkins is not caused b

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

2019-05-21 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 10: (18 comments) Well, it seems that the failure of jenkins

[kudu-CR] [doc] Update the doc of AsyncKuduClient

2019-05-21 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13389 Change subject: [doc] Update the doc of AsyncKuduClient .. [doc] Update the doc of AsyncKuduClient Remove MasterErrorException in doc. Change

[kudu-CR] KUDU-2802 [java] tableExists API optimizations

2019-05-22 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13401 Change subject: KUDU-2802 [java] tableExists API optimizations .. KUDU-2802 [java] tableExists API optimizations Currently the Java client `ta

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

2019-05-22 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#31) 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-22 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#32) 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-22 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 32: (1 comment) Thanks Adar for the test result. I added the inf

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

2019-05-23 Thread Yao Xu (Code Review)
Yao Xu has uploaded a new patch set (#33) 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-2826: We found kudu needs traversing the whole mutation list to find tail to append the new mutations when UPDATE or REINSERT in memrowset. And it cost considerable time when we update

2019-05-23 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13412 ) Change subject: KUDU-2826: We found kudu needs traversing the whole mutation list to find tail to append the new mutations when UPDATE or REINSERT in memrowset. And it cost considerable time when we update the

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

2019-05-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 33: > > The reason for the problem should be that KeyRange is dup

[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 u

[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-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] 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. ...

  1   2   3   >