[GitHub] [maven-resolver] hboutemy commented on a diff in pull request #261: [MRESOLVER-333] Better errors about artifact availability

["hboutemy (via GitHub)" <gi...@apache.org>]

hboutemy commented on code in PR #261:
URL: https://github.com/apache/maven-resolver/pull/261#discussion_r1120066410


##########
maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultArtifactResolver.java:
##########
@@ -382,7 +383,11 @@ private List<ArtifactResult> resolve(
             }
 
             if (local.getFile() != null) {
-                LOGGER.debug("Verifying availability of {} from {}", local.getFile(), remoteRepositories);
+                LOGGER.info(
+                        "Artifact {} is locally present, but unavailable, verifying availability of {} from {}",

Review Comment:
   s/locally present/present in local repository/



##########
maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultArtifactResolver.java:
##########
@@ -382,7 +383,11 @@ private List<ArtifactResult> resolve(
             }
 
             if (local.getFile() != null) {
-                LOGGER.debug("Verifying availability of {} from {}", local.getFile(), remoteRepositories);
+                LOGGER.info(
+                        "Artifact {} is locally present, but unavailable, verifying availability of {} from {}",

Review Comment:
   s/but unavailable/but cached from a repository id that is unavailable in current build context/



##########
maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultArtifactResolver.java:
##########
@@ -382,7 +383,11 @@ private List<ArtifactResult> resolve(
             }
 
             if (local.getFile() != null) {
-                LOGGER.debug("Verifying availability of {} from {}", local.getFile(), remoteRepositories);
+                LOGGER.info(
+                        "Artifact {} is locally present, but unavailable, verifying availability of {} from {}",

Review Comment:
   s/verifying availability of {} from {}/ that downloads from {}/



-- 
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: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org