You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Yao Xu (Code Review)" <ge...@cloudera.org> on 2019/08/12 13:07:28 UTC

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows 

Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14046


Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................

KUDU-2917 Split a tablet into primary key ranges by number of rows

Since we implemented KUDU-2437 and KUDU-2670, the spark job can read
data inside the tablet in parallel.

However, we found in actual use that splitting key range by size may
cause the spark task to read long tails. This issue is caused by the
encoding and compression of column-wise.

So I added the ability to split key range by number of rows. As long
as one of the 'target_chunk_size' or 'target_chunk_rows' is satisfied,
a new key range will be created.

Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
---
M src/kudu/common/common.proto
M src/kudu/common/key_range.cc
M src/kudu/common/key_range.h
M src/kudu/tablet/mock-rowsets.h
M src/kudu/tablet/rowset_info.cc
M src/kudu/tablet/rowset_info.h
M src/kudu/tablet/tablet-test.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet.h
M src/kudu/tserver/tablet_server-test.cc
M src/kudu/tserver/tablet_service.cc
M src/kudu/tserver/tserver.proto
12 files changed, 266 insertions(+), 133 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/14046/1
-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 1
Gerrit-Owner: Yao Xu <oc...@gmail.com>

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14046 )

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................


Patch Set 6:

Looks like this patch needs a rebase (but it conflicts with master right now).


-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 6
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Comment-Date: Tue, 12 Nov 2019 08:13:58 +0000
Gerrit-HasComments: No

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Yao Xu (Code Review)" <ge...@cloudera.org>.
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/14046 )

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................


Patch Set 6:

> It looks like CountLiveRows will return a negative number in some
 > cases. I will continue to address this issue.

Looks like a bug with CountLiveRows. I tried the fix and ran it locally 1000 times without a problem.


-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 6
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Comment-Date: Tue, 13 Aug 2019 09:55:09 +0000
Gerrit-HasComments: No

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Yao Xu (Code Review)" <ge...@cloudera.org>.
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/14046

to look at the new patch set (#5).

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................

KUDU-2917 Split a tablet into primary key ranges by number of rows

Since we implemented KUDU-2437 and KUDU-2670, the spark job can read
data inside the tablet in parallel.

However, we found in actual use that splitting key range by size may
cause the spark task to read long tails. This issue is caused by the
encoding and compression of column-wise.

So I added the ability to split key range by number of rows. As long
as one of the 'target_chunk_size' or 'target_chunk_rows' is satisfied,
a new key range will be created.

Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
---
M src/kudu/common/common.proto
M src/kudu/common/key_range.cc
M src/kudu/common/key_range.h
M src/kudu/tablet/delta_tracker.cc
M src/kudu/tablet/delta_tracker.h
M src/kudu/tablet/deltamemstore.cc
M src/kudu/tablet/diskrowset.cc
M src/kudu/tablet/mock-rowsets.h
M src/kudu/tablet/rowset_info.cc
M src/kudu/tablet/rowset_info.h
M src/kudu/tablet/tablet-test.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet.h
M src/kudu/tserver/tablet_server-test.cc
M src/kudu/tserver/tablet_service.cc
M src/kudu/tserver/tserver.proto
16 files changed, 278 insertions(+), 151 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/14046/5
-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 5
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14046 )

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................


Patch Set 6:

> > It looks like CountLiveRows will return a negative number in some
>  > cases. I will continue to address this issue.
> 
> Looks like a bug with CountLiveRows. I tried the fix and ran it locally 1000 times without a problem.

Could you pull the fix out into a separate patch (with a unit test), and cc He Lifu for review?


-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 6
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Comment-Date: Tue, 13 Aug 2019 19:02:00 +0000
Gerrit-HasComments: No

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Yao Xu (Code Review)" <ge...@cloudera.org>.
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/14046

to look at the new patch set (#4).

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................

KUDU-2917 Split a tablet into primary key ranges by number of rows

Since we implemented KUDU-2437 and KUDU-2670, the spark job can read
data inside the tablet in parallel.

However, we found in actual use that splitting key range by size may
cause the spark task to read long tails. This issue is caused by the
encoding and compression of column-wise.

So I added the ability to split key range by number of rows. As long
as one of the 'target_chunk_size' or 'target_chunk_rows' is satisfied,
a new key range will be created.

Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
---
M src/kudu/common/common.proto
M src/kudu/common/key_range.cc
M src/kudu/common/key_range.h
M src/kudu/tablet/deltamemstore.cc
M src/kudu/tablet/diskrowset.cc
M src/kudu/tablet/mock-rowsets.h
M src/kudu/tablet/rowset_info.cc
M src/kudu/tablet/rowset_info.h
M src/kudu/tablet/tablet-test.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet.h
M src/kudu/tserver/tablet_server-test.cc
M src/kudu/tserver/tablet_service.cc
M src/kudu/tserver/tserver.proto
14 files changed, 274 insertions(+), 139 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/14046/4
-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 4
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Yao Xu (Code Review)" <ge...@cloudera.org>.
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/14046 )

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................


Patch Set 2:

It looks like CountLiveRows will return a negative number in some cases. I will continue to address this issue.


-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 2
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Comment-Date: Mon, 12 Aug 2019 14:39:12 +0000
Gerrit-HasComments: No

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Yao Xu (Code Review)" <ge...@cloudera.org>.
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/14046 )

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................


Patch Set 6:

(1 comment)

> (1 comment)
 > 
 > Can you quantify the performance impact using a benchmark?
 > 
 > I'm also curious whether you looked into making the existing
 > chunk-by-size heuristic more accurate by considering encoded size,
 > factoring in deltas, or something along those lines.

Sure, I'm going to implement a SplitKeyRange benchmark.

I have considered adding a decode data size to the DiskRowSet to make splitKeyRange better. However, it is still difficult to resolve the case where column data is null. So I first implemented the feature of splitting key range by the number of rows.

http://gerrit.cloudera.org:8080/#/c/14046/4/src/kudu/tablet/deltamemstore.cc
File src/kudu/tablet/deltamemstore.cc:

http://gerrit.cloudera.org:8080/#/c/14046/4/src/kudu/tablet/deltamemstore.cc@124
PS4, Line 124: 
             :   if (update.is_delete()) {
             :    
> The DMS doesn't allow REINSERTs. Maybe you can enforce that via DCHECK?
You're right. I misremembered. :(



-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 6
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Comment-Date: Tue, 13 Aug 2019 09:47:39 +0000
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14046 )

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................


Patch Set 4:

(1 comment)

Can you quantify the performance impact using a benchmark?

I'm also curious whether you looked into making the existing chunk-by-size heuristic more accurate by considering encoded size, factoring in deltas, or something along those lines.

http://gerrit.cloudera.org:8080/#/c/14046/4/src/kudu/tablet/deltamemstore.cc
File src/kudu/tablet/deltamemstore.cc:

http://gerrit.cloudera.org:8080/#/c/14046/4/src/kudu/tablet/deltamemstore.cc@124
PS4, Line 124:  else if (update.is_reinsert()) {
             :     deleted_row_count_.IncrementBy(-1);
             :   }
The DMS doesn't allow REINSERTs. Maybe you can enforce that via DCHECK?



-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 4
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Comment-Date: Tue, 13 Aug 2019 08:14:05 +0000
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Yao Xu (Code Review)" <ge...@cloudera.org>.
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/14046

to look at the new patch set (#3).

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................

KUDU-2917 Split a tablet into primary key ranges by number of rows

Since we implemented KUDU-2437 and KUDU-2670, the spark job can read
data inside the tablet in parallel.

However, we found in actual use that splitting key range by size may
cause the spark task to read long tails. This issue is caused by the
encoding and compression of column-wise.

So I added the ability to split key range by number of rows. As long
as one of the 'target_chunk_size' or 'target_chunk_rows' is satisfied,
a new key range will be created.

Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
---
M src/kudu/common/common.proto
M src/kudu/common/key_range.cc
M src/kudu/common/key_range.h
M src/kudu/tablet/diskrowset.cc
M src/kudu/tablet/mock-rowsets.h
M src/kudu/tablet/rowset_info.cc
M src/kudu/tablet/rowset_info.h
M src/kudu/tablet/tablet-test.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet.h
M src/kudu/tserver/tablet_server-test.cc
M src/kudu/tserver/tablet_service.cc
M src/kudu/tserver/tserver.proto
13 files changed, 271 insertions(+), 138 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/14046/3
-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 3
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Yao Xu (Code Review)" <ge...@cloudera.org>.
Hello Tidy Bot, Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/14046

to look at the new patch set (#2).

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................

KUDU-2917 Split a tablet into primary key ranges by number of rows

Since we implemented KUDU-2437 and KUDU-2670, the spark job can read
data inside the tablet in parallel.

However, we found in actual use that splitting key range by size may
cause the spark task to read long tails. This issue is caused by the
encoding and compression of column-wise.

So I added the ability to split key range by number of rows. As long
as one of the 'target_chunk_size' or 'target_chunk_rows' is satisfied,
a new key range will be created.

Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
---
M src/kudu/common/common.proto
M src/kudu/common/key_range.cc
M src/kudu/common/key_range.h
M src/kudu/tablet/mock-rowsets.h
M src/kudu/tablet/rowset_info.cc
M src/kudu/tablet/rowset_info.h
M src/kudu/tablet/tablet-test.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet.h
M src/kudu/tserver/tablet_server-test.cc
M src/kudu/tserver/tablet_service.cc
M src/kudu/tserver/tserver.proto
12 files changed, 269 insertions(+), 137 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/14046/2
-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 2
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)

[kudu-CR] KUDU-2917 Split a tablet into primary key ranges by number of rows

Posted by "Yao Xu (Code Review)" <ge...@cloudera.org>.
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/14046

to look at the new patch set (#6).

Change subject: KUDU-2917 Split a tablet into primary key ranges by number of rows
......................................................................

KUDU-2917 Split a tablet into primary key ranges by number of rows

Since we implemented KUDU-2437 and KUDU-2670, the spark job can read
data inside the tablet in parallel.

However, we found in actual use that splitting key range by size may
cause the spark task to read long tails. This issue is caused by the
encoding and compression of column-wise.

So I added the ability to split key range by number of rows. As long
as one of the 'target_chunk_size' or 'target_chunk_rows' is satisfied,
a new key range will be created.

Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
---
M src/kudu/common/common.proto
M src/kudu/common/key_range.cc
M src/kudu/common/key_range.h
M src/kudu/tablet/delta_tracker.cc
M src/kudu/tablet/delta_tracker.h
M src/kudu/tablet/deltamemstore.cc
M src/kudu/tablet/diskrowset.cc
M src/kudu/tablet/mock-rowsets.h
M src/kudu/tablet/rowset_info.cc
M src/kudu/tablet/rowset_info.h
M src/kudu/tablet/tablet-test.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet.h
M src/kudu/tserver/tablet_server-test.cc
M src/kudu/tserver/tablet_service.cc
M src/kudu/tserver/tserver.proto
16 files changed, 279 insertions(+), 152 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/14046/6
-- 
To view, visit http://gerrit.cloudera.org:8080/14046
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib859c615743ebf7c3b481dfe1bd428769b02f0d7
Gerrit-Change-Number: 14046
Gerrit-PatchSet: 6
Gerrit-Owner: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>