-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support --unix_digest_hash_attribute_name
in repository cache
#18221
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
1518e04
to
1b3cfe3
Compare
src/main/java/com/google/devtools/build/lib/bazel/repository/cache/RepositoryCache.java
Outdated
Show resolved
Hide resolved
--unix_digest_hash_attribute_name
in repository cache
src/main/java/com/google/devtools/build/lib/bazel/repository/cache/RepositoryCache.java
Outdated
Show resolved
Hide resolved
46c2c25
to
e188737
Compare
740ccee
to
6d01750
Compare
src/main/java/com/google/devtools/build/lib/bazel/repository/cache/RepositoryCache.java
Show resolved
Hide resolved
src/test/java/com/google/devtools/build/lib/bazel/repository/cache/RepositoryCacheTest.java
Outdated
Show resolved
Hide resolved
src/main/java/com/google/devtools/build/lib/vfs/inmemoryfs/InMemoryFileInfo.java
Outdated
Show resolved
Hide resolved
@coeuvre Thanks for that flag! Does this look better? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
We extend `RepositoryCache` and `DownloadManager` to support pulling checksums from extended attributes. In some example builds, I observe a 5x speedup in repository fetching: https://github.com/sushain97/bazel-faster-fetch-repro. I would like to make `xattrProvider` not-nullable in any context. However, it's not immediately clear how I could thread access to one in the `BazelPackageLoader` context: https://github.com/sushain97/bazel/blob/1518e04579e6a1c966eb8fbf6d65603e0a1d41d5/src/main/java/com/google/devtools/build/lib/skyframe/packages/BazelPackageLoader.java#L108-L110. Open to any suggestions. Closes bazelbuild#18221. PiperOrigin-RevId: 529730007 Change-Id: Ic06b4585988a6b5c09f61cee68daf1dcdf3e46fc
We extend
RepositoryCache
andDownloadManager
to support pulling checksums from extended attributes. In some example builds, I observe a 5x speedup in repository fetching: https://github.com/sushain97/bazel-faster-fetch-repro.I would like to make
xattrProvider
not-nullable in any context. However, it's not immediately clear how I could thread access to one in theBazelPackageLoader
context: https://github.com/sushain97/bazel/blob/1518e04579e6a1c966eb8fbf6d65603e0a1d41d5/src/main/java/com/google/devtools/build/lib/skyframe/packages/BazelPackageLoader.java#L108-L110. Open to any suggestions.