helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/14964 )
Change subject: KUDU-3007. Support building Kudu on aarch64 platform ...................................................................... Patch Set 18: > Could you separate all third-party files imported as-is into a > separate change list and then rebase this patch on top of that? > > Having its own changelist for those third-party files will help to > separate the changes in the Kudu's and third-party code to support > Kudu building and running on aarch64. Agree with Alexey's comment. As we discussed before, small,independent and incremental patches will be more conducive to review. //By the way, good job @liusheng :) -- To view, visit http://gerrit.cloudera.org:8080/14964 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2953519c5d28de17e6b2bb7094abab0c1cd12c97 Gerrit-Change-Number: 14964 Gerrit-PatchSet: 18 Gerrit-Owner: liusheng <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: RuiChen <[email protected]> Gerrit-Reviewer: helifu <[email protected]> Gerrit-Reviewer: liusheng <[email protected]> Gerrit-Comment-Date: Wed, 06 May 2020 12:05:30 +0000 Gerrit-HasComments: No
