[kudu-CR] [tablet] Fixed the bug of DeltaTracker::CountDeletedRows

2019-08-20 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, helifu, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14061 to look at the new patch set (#10). Change subject: [tablet] Fixed the bug of DeltaTracker::CountDeletedRows

[kudu-CR] [tablet] Fixed the bug of DeltaTracker::CountDeletedRows

2019-08-20 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/14061 ) Change subject: [tablet] Fixed the bug of DeltaTracker::CountDeletedRows .. Patch Set 10: (1 comment)

[kudu-CR] [docs] Remove the duplicate word

2019-08-01 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13977 Change subject: [docs] Remove the duplicate word .. [docs] Remove the duplicate word Change-Id: Ie988c1e7e635449c4ff5ca40170d628b9b3a1a8e ---

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-28 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13908 ) Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client .. Patch Set 9: (2 comments)

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-28 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13908 to look at the new patch set (#9). Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-26 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13908 ) Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client .. Patch Set 8: (4 comments)

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-26 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13908 to look at the new patch set (#7). Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-26 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13908 to look at the new patch set (#8). Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client

[kudu-CR] KUDU-2823 [java client] Support setting dimension for the newly created tablet

2019-07-17 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13875 ) Change subject: KUDU-2823 [java client] Support setting dimension for the newly created tablet .. Patch Set 2: (2 comments)

[kudu-CR] KUDU-2823 [java client] Support setting dimension for the newly created tablet

2019-07-17 Thread Yao Xu (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13875 to look at the new patch set (#3). Change subject: KUDU-2823 [java client] Support setting dimension for the newly created tablet

[kudu-CR] KUDU-2823 [java client] Support setting dimension for the newly created tablet

2019-07-17 Thread Yao Xu (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13875 to look at the new patch set (#4). Change subject: KUDU-2823 [java client] Support setting dimension for the newly created tablet

[kudu-CR] KUDU-2823 [java client] Support setting dimension for the newly created tablet

2019-07-17 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13875 ) Change subject: KUDU-2823 [java client] Support setting dimension for the newly created tablet .. Patch Set 4: (2 comments)

[kudu-CR] KUDU-2823 [java client] Support setting dimension for the newly created tablet

2019-07-18 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13875 ) Change subject: KUDU-2823 [java client] Support setting dimension for the newly created tablet .. Patch Set 4: (2 comments)

[kudu-CR] [docs]: Delete invalid link

2019-07-19 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13881 ) Change subject: [docs]: Delete invalid link .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2823 [java client] Support setting dimension for the newly created tablet

2019-07-19 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13875 ) Change subject: KUDU-2823 [java client] Support setting dimension for the newly created tablet .. Patch Set 4: (1 comment)

[kudu-CR] Deprecated TabletLocationsPB.ReplicaPB message

2019-07-24 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13908 Change subject: Deprecated TabletLocationsPB.ReplicaPB message .. Deprecated TabletLocationsPB.ReplicaPB message According to the comments of

[kudu-CR] Deprecated TabletLocationsPB.ReplicaPB message

2019-07-24 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13908 ) Change subject: Deprecated TabletLocationsPB.ReplicaPB message .. Patch Set 1: > (1 comment) It seems like this. :( I think I can make some

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-24 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/13908 to look at the new patch set (#2). Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-24 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13908 to look at the new patch set (#3). Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client

[kudu-CR] KUDU-2828 [java client] Support setting dimension for the newly created tablet

2019-07-17 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13875 Change subject: KUDU-2828 [java client] Support setting dimension for the newly created tablet .. KUDU-2828 [java client] Support setting

[kudu-CR] KUDU-2823 [java client] Support setting dimension for the newly created tablet

2019-07-17 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13875 to look at the new patch set (#2). Change subject: KUDU-2823 [java client] Support setting dimension for the newly created tablet

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-25 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13908 ) Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client .. Patch Set 4: (3 comments)

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-24 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13908 ) Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client .. Patch Set 3: (1 comment)

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-25 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13908 to look at the new patch set (#4). Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-26 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13908 to look at the new patch set (#6). Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-26 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13908 ) Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client .. Patch Set 6: (1 comment)

[kudu-CR] [client] Deprecated TabletLocationsPB.ReplicaPB message in client

2019-07-26 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13908 to look at the new patch set (#5). Change subject: [client] Deprecated TabletLocationsPB.ReplicaPB message in client

[kudu-CR] KUDU-2823 [java client] Support setting dimension for the newly created tablet

2019-07-21 Thread Yao Xu (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13875 to look at the new patch set (#5). Change subject: KUDU-2823 [java client] Support setting dimension for the newly created tablet

[kudu-CR] KUDU-2823 [java client] Support setting dimension for the newly created tablet

2019-07-21 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13875 ) Change subject: KUDU-2823 [java client] Support setting dimension for the newly created tablet .. Patch Set 5: (1 comment)

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

2019-06-13 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: > Emm, it looks like heartbeat failed. I'll try to fix it. I

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

2019-07-10 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#15). Change subject: KUDU-2823 Place tablet replicas based on dimension

[kudu-CR] KUDU-2847: Optimize iteration over selection vector in SerializeRowBlock

2019-07-10 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13721 ) Change subject: KUDU-2847: Optimize iteration over selection vector in SerializeRowBlock .. Patch Set 2: (2 comments)

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

2019-07-10 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 dimension .. Patch Set 15: (12 comments) > (12 comments) > > Just passing

[kudu-CR] KUDU-2722 (table level): Support table extraConfig 'write enabled' conf

2019-07-10 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13796 ) Change subject: KUDU-2722 (table level): Support table extraConfig 'write_enabled' conf .. Patch Set 1: I'm curious about how to implement

[kudu-CR] KUDU-1260 Add test for prefetching on java scanner

2019-11-04 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14631 Change subject: KUDU-1260 Add test for prefetching on java scanner .. KUDU-1260 Add test for prefetching on java scanner Change-Id:

[kudu-CR] KUDU-1260 Add test for prefetching on java scanner

2019-11-04 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/14631 ) Change subject: KUDU-1260 Add test for prefetching on java scanner .. Patch Set 1: Hi, adar, grant. I for prefetching scanner added some

[kudu-CR] [spark] Add prefetching option to kudu-spark

2019-11-01 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/14598 ) Change subject: [spark] Add prefetching option to kudu-spark .. Patch Set 1: > (1 comment) Maybe we can use the existing testcase to test

[kudu-CR] [spark] Add prefetching option to kudu-spark

2019-11-01 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/14598 ) Change subject: [spark] Add prefetching option to kudu-spark .. Patch Set 1: > (1 comment) > > Thanks for finding and fixing this bug.

[kudu-CR] [spark] Add prefetching option to kudu-spark

2019-10-30 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14598 Change subject: [spark] Add prefetching option to kudu-spark .. [spark] Add prefetching option to kudu-spark We have already supported the

[kudu-CR] KUDU-1260 Add test for prefetching on java scanner

2019-11-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/14631 to look at the new patch set (#2). Change subject: KUDU-1260 Add test for prefetching on java scanner

[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

<    1   2   3