Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14652 )
Change subject: KUDU-2162 Expose stats about scan filters ...................................................................... Patch Set 4: > > Just realized that this is a continuation of Will's patch from > > https://gerrit.cloudera.org/c/8375. > > > > Could you restore that gerrit change and upload a new version of > > it? That way we can continue the review from where Will left off, > > and you can explicitly mark all of Todd's feedback as addressed. > > Mechanically, all you need to do is ensure that the Change-Id is > > the same in the commit as it was in Will's patch. After that, just > > push to gerrit and gerrit will allow a second author of the patch. > > Thanks for the review. I tried using the same change id as the patch from > https://gerrit.cloudera.org/c/8375 but I get an error when I try to push > > ! [remote rejected] HEAD -> refs/for/master (change > http://gerrit.cloudera.org:8080/8375 closed) > > I think this might be because the patch was abandoned, can you please reopen > the change, I don't have permissions. I reopened it, try again. -- To view, visit http://gerrit.cloudera.org:8080/14652 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I127b96a30467ea0abcaf6ed5b4d72df66b2dec55 Gerrit-Change-Number: 14652 Gerrit-PatchSet: 4 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sat, 09 Nov 2019 22:47:43 +0000 Gerrit-HasComments: No
