assignUser commented on PR #38195:
URL: https://github.com/apache/arrow/pull/38195#issuecomment-1759882061

   Thanks for testing @jonkeane !
   
   > but setting this to use more cores might be nice + possible then.
   
   In the follow up #38236 I automatically set not_cran when we are in a dev 
build, which should make the dev experience on especially linux but overall 
much smoother as you also won't have to fiddle with allow lists or such to get 
the binaries. 
   
   I am a bit hesitant to remove the 2 core cap by default for release builds 
as we could still have to build from source on cran (BDR special checks? 
Condition for use of binaries ...). Mac users will get the cran binary anyway 
and through the allow list a majority of the linux users will also get 
binaries, so I think the reward doesn't justify the risk of a policy violation. 
(I also think that the 2 core think is a philosophical stance not necessarily 
limited to builds on cran, at least going by recent r dev ml threads...)
   
   > but the linking step seemed to use all of my cores
   
   :scream: I didn't change anything there so I am pretty sure that was already 
happening (on linuc), so it looks like you are right and the short burst is 
within tolerance? 
   
   > I think my local 11.7 build fails because it's mixing homebrew and bundled 
dependencies
   
   Hm that should not be a problem normally though unless there is some version 
mismatch? The 10.13 runner also has brew and even has the deps installed but no 
problem arises... as this is a definitive dev use case I think it's ok to have 
people manually set the deps source var if the problem happens. Could you try 
with `ARROW_DEPENDENCY_SOURCE= BUNDELD`?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to