liusheng 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. That sounds great. it is better to support aarch64 in upstream crcutil . thank you :) -- 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: Thu, 19 Dec 2019 02:58:49 +0000 Gerrit-HasComments: No
