Joe McDonnell has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24496 )

Change subject: IMPALA-15109: Add ninja to the toolchain
......................................................................


Patch Set 1:

> Looks good, although the Kudu build doesn't yet use the freshly
 > built Ninja binary.

The main benefit of ninja is being able to limit link concurrency, but Kudu 
currently doesn't set those limits for its thirdparty LLVM. I'll file a 
follow-up ticket for that.


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

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ied3c4a200473f275dbd459075d6d3d8ec43eea84
Gerrit-Change-Number: 24496
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Tue, 14 Jul 2026 22:22:53 +0000
Gerrit-HasComments: No

Reply via email to