Repository: maven Updated Branches: refs/heads/master 14bff2c5c -> 8a1b6359f
[MNG-5457] Show repository id when downloading or uploading from/to a remote repository Project: http://git-wip-us.apache.org/repos/asf/maven/repo Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/8a1b6359 Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/8a1b6359 Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/8a1b6359 Branch: refs/heads/master Commit: 8a1b6359f31178ce804018cfb524e72f1178028a Parents: 14bff2c Author: Michael Osipov <micha...@apache.org> Authored: Fri Jul 15 23:26:56 2016 +0200 Committer: Michael Osipov <micha...@apache.org> Committed: Sat Dec 17 20:31:05 2016 +0100 ---------------------------------------------------------------------- .../transfer/AbstractMavenTransferListener.java | 32 +++++++++++++------- .../transfer/Slf4jMavenTransferListener.java | 31 +++++++++++++------ 2 files changed, 42 insertions(+), 21 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/maven/blob/8a1b6359/maven-embedder/src/main/java/org/apache/maven/cli/transfer/AbstractMavenTransferListener.java ---------------------------------------------------------------------- diff --git a/maven-embedder/src/main/java/org/apache/maven/cli/transfer/AbstractMavenTransferListener.java b/maven-embedder/src/main/java/org/apache/maven/cli/transfer/AbstractMavenTransferListener.java index e72aa47..2286510 100644 --- a/maven-embedder/src/main/java/org/apache/maven/cli/transfer/AbstractMavenTransferListener.java +++ b/maven-embedder/src/main/java/org/apache/maven/cli/transfer/AbstractMavenTransferListener.java @@ -23,7 +23,6 @@ import java.io.PrintStream; import java.text.DecimalFormat; import java.text.DecimalFormatSymbols; import java.util.Locale; - import org.apache.commons.lang3.Validate; import org.eclipse.aether.transfer.AbstractTransferListener; import org.eclipse.aether.transfer.TransferCancelledException; @@ -217,9 +216,15 @@ public abstract class AbstractMavenTransferListener public void transferInitiated( TransferEvent event ) { String type = event.getRequestType() == TransferEvent.RequestType.PUT ? "Uploading" : "Downloading"; + String direction = event.getRequestType() == TransferEvent.RequestType.PUT ? "to" : "from"; TransferResource resource = event.getResource(); - out.println( type + ": " + resource.getRepositoryUrl() + resource.getResourceName() ); + StringBuilder message = new StringBuilder(); + message.append( type ).append( ' ' ).append( direction ).append( ' ' ).append( resource.getRepositoryId() ); + message.append( ": " ); + message.append( resource.getRepositoryUrl() ).append( resource.getResourceName() ); + + out.println( message.toString() ); } @Override @@ -227,30 +232,35 @@ public abstract class AbstractMavenTransferListener throws TransferCancelledException { TransferResource resource = event.getResource(); - out.println( "[WARNING] " + event.getException().getMessage() + " for " + resource.getRepositoryUrl() - + resource.getResourceName() ); + out.println( "[WARNING] " + event.getException().getMessage() + " from " + resource.getRepositoryId() + " for " + + resource.getRepositoryUrl() + resource.getResourceName() ); } @Override public void transferSucceeded( TransferEvent event ) { + String type = ( event.getRequestType() == TransferEvent.RequestType.PUT ? "Uploaded" : "Downloaded" ); + String direction = event.getRequestType() == TransferEvent.RequestType.PUT ? "to" : "from"; + TransferResource resource = event.getResource(); long contentLength = event.getTransferredBytes(); - FileSizeFormat format = new FileSizeFormat( Locale.ENGLISH ); - String type = ( event.getRequestType() == TransferEvent.RequestType.PUT ? "Uploaded" : "Downloaded" ); - String len = format.format( contentLength ); - String throughput = ""; + StringBuilder message = new StringBuilder(); + message.append( type ).append( ' ' ).append( direction ).append( ' ' ).append( resource.getRepositoryId() ); + message.append( ": " ); + message.append( resource.getRepositoryUrl() ).append( resource.getResourceName() ); + message.append( " (" ).append( format.format( contentLength ) ); + long duration = System.currentTimeMillis() - resource.getTransferStartTime(); if ( duration > 0L ) { double bytesPerSecond = contentLength / ( duration / 1000.0 ); - throughput = " at " + format.format( (long) bytesPerSecond ) + "/s"; + message.append( " at " ).append( format.format( (long) bytesPerSecond ) ).append( "/s" ); } - out.println( type + ": " + resource.getRepositoryUrl() + resource.getResourceName() + " (" + len - + throughput + ")" ); + message.append( ')' ); + out.println( message.toString() ); } } http://git-wip-us.apache.org/repos/asf/maven/blob/8a1b6359/maven-embedder/src/main/java/org/apache/maven/cli/transfer/Slf4jMavenTransferListener.java ---------------------------------------------------------------------- diff --git a/maven-embedder/src/main/java/org/apache/maven/cli/transfer/Slf4jMavenTransferListener.java b/maven-embedder/src/main/java/org/apache/maven/cli/transfer/Slf4jMavenTransferListener.java index 5bfb7b4..5d1144a 100644 --- a/maven-embedder/src/main/java/org/apache/maven/cli/transfer/Slf4jMavenTransferListener.java +++ b/maven-embedder/src/main/java/org/apache/maven/cli/transfer/Slf4jMavenTransferListener.java @@ -50,9 +50,15 @@ public class Slf4jMavenTransferListener public void transferInitiated( TransferEvent event ) { String type = event.getRequestType() == TransferEvent.RequestType.PUT ? "Uploading" : "Downloading"; + String direction = event.getRequestType() == TransferEvent.RequestType.PUT ? "to" : "from"; TransferResource resource = event.getResource(); - out.info( type + ": " + resource.getRepositoryUrl() + resource.getResourceName() ); + StringBuilder message = new StringBuilder(); + message.append( type ).append( ' ' ).append( direction ).append( ' ' ).append( resource.getRepositoryId() ); + message.append( ": " ); + message.append( resource.getRepositoryUrl() ).append( resource.getResourceName() ); + + out.info( message.toString() ); } @Override @@ -60,30 +66,35 @@ public class Slf4jMavenTransferListener throws TransferCancelledException { TransferResource resource = event.getResource(); - out.warn( event.getException().getMessage() + " for " + resource.getRepositoryUrl() - + resource.getResourceName() ); + out.warn( event.getException().getMessage() + " from " + resource.getRepositoryId() + " for " + + resource.getRepositoryUrl() + resource.getResourceName() ); } @Override public void transferSucceeded( TransferEvent event ) { + String type = ( event.getRequestType() == TransferEvent.RequestType.PUT ? "Uploaded" : "Downloaded" ); + String direction = event.getRequestType() == TransferEvent.RequestType.PUT ? "to" : "from"; + TransferResource resource = event.getResource(); long contentLength = event.getTransferredBytes(); - FileSizeFormat format = new FileSizeFormat( Locale.ENGLISH ); - String type = ( event.getRequestType() == TransferEvent.RequestType.PUT ? "Uploaded" : "Downloaded" ); - String len = format.format( contentLength ); - String throughput = ""; + StringBuilder message = new StringBuilder(); + message.append( type ).append( ' ' ).append( direction ).append( ' ' ).append( resource.getRepositoryId() ); + message.append( ": " ); + message.append( resource.getRepositoryUrl() ).append( resource.getResourceName() ); + message.append( " (" ).append( format.format( contentLength ) ); + long duration = System.currentTimeMillis() - resource.getTransferStartTime(); if ( duration > 0L ) { double bytesPerSecond = contentLength / ( duration / 1000.0 ); - throughput = " at " + format.format( (long) bytesPerSecond ) + "/s"; + message.append( " at " ).append( format.format( (long) bytesPerSecond ) ).append( "/s" ); } - out.info( type + ": " + resource.getRepositoryUrl() + resource.getResourceName() + " (" + len - + throughput + ")" ); + message.append( ')' ); + out.info( message.toString() ); } }