commit 3ca9ce13423393b90564710670b09486955383af
author Fabian Meumertzheim <fabian@meumertzhe.im> Tue Nov 18 20:53:30 2025 -0800
committer Copybara-Service <copybara-worker@google.com> Tue Nov 18 20:54:53 2025 -0800
tree 40af08a7d432071605c5b638bbea28ffb7dc72c9
parent 491f2524883f318dd7293f59de85c025a2148607

Make remote repo contents cache less spammy

Don't print a message when it's successful. Users can always look under `external` to verify which repo came from the cache.

Closes #27699.

PiperOrigin-RevId: 834096735
Change-Id: I3916fb240218a6b68ecf48417142b998ca281598

src/main/java/com/google/devtools/build/lib/bazel/repository/RepositoryFetchFunction.java

diff --git a/src/main/java/com/google/devtools/build/lib/bazel/repository/RepositoryFetchFunction.java b/src/main/java/com/google/devtools/build/lib/bazel/repository/RepositoryFetchFunction.java
index 6b71892..d54bfa4 100644
--- a/src/main/java/com/google/devtools/build/lib/bazel/repository/RepositoryFetchFunction.java
+++ b/src/main/java/com/google/devtools/build/lib/bazel/repository/RepositoryFetchFunction.java
@@ -174,7 +174,6 @@
           RepositoryUtils.getExternalRepositoryDirectory(directories)
               .getRelative(repositoryName.getName());
 
-
       RepoDefinition repoDefinition;
       switch ((RepoDefinitionValue) env.getValue(RepoDefinitionValue.key(repositoryName))) {
         case null -> {
@@ -258,10 +257,6 @@
           try {
             if (remoteRepoContentsCache.lookupCache(
                 repositoryName, repoRoot, digestWriter.predeclaredInputHash, env.getListener())) {
-              env.getListener()
-                  .handle(
-                      Event.debug(
-                          "Got %s from the remote repo contents cache".formatted(repositoryName)));
               return new RepositoryDirectoryValue.Success(
                   Root.fromPath(repoRoot), excludeRepoFromVendoring);
             }