Yingchun Lai has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20195 )

Change subject: KUDU-3476: Make replica placement range and table aware
......................................................................


Patch Set 2:

> Patch Set 2:
>
> > Patch Set 2:
> >
> > > Could you please separate the original commit and the updating of
> >  > the release note? Then the commits on the two branches can be kept
> >  > in one-to-one correspondence.
> >  > Besides, run 'git cherry-pick' with "-x" option, so the message
> >  > would contain the UUID of the original commit.
> >
> > +1
>
> The UUID of the original commit won't show since I had to manually resolve a 
> conflict when cherry picking. I can manually add the UUID of the original 
> commit if you would like. I'll go ahead and separate this patch and the 
> update of the release notes.

It's acceptable, thanks!


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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.17.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I9caeb8d5547e946bfeb152a99e1ec034c3fa0a0f
Gerrit-Change-Number: 20195
Gerrit-PatchSet: 2
Gerrit-Owner: Mahesh Reddy <mre...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <mre...@cloudera.com>
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Comment-Date: Fri, 21 Jul 2023 02:47:31 +0000
Gerrit-HasComments: No

Reply via email to