Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14873 )
Change subject: [KUDU-3007] Add patch for crcutil to support aarch64 platform ...................................................................... Patch Set 6: > Can this be merged ? I just clicked the "Rebase" and "+2" lost :) We could, except that at the same time that you've been doing this work, we've been working to publish a real crcutil repo (rather than a fork in my own github account). That repo will include this patch in it so that Kudu needn't maintain it downstream. Once the repo is made public, I'll publish a patch to update Kudu's crcutil dependency to a tarball including those patches. Which means we won't actually need this particular patch, as it'll be available in "upstream" crcutil. -- To view, visit http://gerrit.cloudera.org:8080/14873 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3a95504b52407d3e213ff76bf9ebcd960ae4fde3 Gerrit-Change-Number: 14873 Gerrit-PatchSet: 6 Gerrit-Owner: liusheng <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: liusheng <[email protected]> Gerrit-Comment-Date: Wed, 18 Dec 2019 01:39:27 +0000 Gerrit-HasComments: No
