neilconway opened a new pull request, #26598:
URL: https://github.com/apache/datafusion/pull/26598

   ## Which issue does this PR close?
   
   - N/A
   
   ## Rationale for this change
   
   For every changed crate, `cargo-semver-checks` compiles two versions of it 
(baseline version vs. PR version). By default, each build uses its own target 
directory. For example, a PR touching three crates will result in six 
from-scratch compilations.
   
   We can improve this by setting `CARGO_TARGET_DIR`, so that all of these 
builds use a shared target directory. This means that unmodified dependencies 
need only be built once.
   
   This improves semver check runtime by ~1.8-3x, depending on the nature of 
the PR, and also reduces disk space consumption.
   
   ## What changes are included in this PR?
   
   See above.
   
   ## What is the testing strategy for this PR?
   
   Tested locally, dev tooling change only.
   
   ## Are there any user-facing changes?
   
   No.
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to