You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicecomb.apache.org by li...@apache.org on 2018/06/07 12:27:41 UTC

[incubator-servicecomb-java-chassis] branch master updated: [SCB-649] when port can't been listened, log warn and show more detail message (#755)

This is an automated email from the ASF dual-hosted git repository.

liubao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-servicecomb-java-chassis.git


The following commit(s) were added to refs/heads/master by this push:
     new ef349bb  [SCB-649] when port can't been listened,log warn and show more detail message (#755)
ef349bb is described below

commit ef349bb295150d7f68b1d09d9f4069d08e8cdc41
Author: weichao <30...@users.noreply.github.com>
AuthorDate: Thu Jun 7 20:27:35 2018 +0800

    [SCB-649] when port can't been listened,log warn and show more detail message (#755)
    
    * [SCB-649] when port can't been listened,log warn and show more detail message
    
    * [SCB-649] when port can't been listened,log warn and show more detail message,modify
    
    * [SCB-649] when port can't been listened,log warn and show more detail message,modified
---
 .../apache/servicecomb/transport/rest/vertx/VertxRestTransport.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/VertxRestTransport.java b/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/VertxRestTransport.java
index 008792c..8ec17c1 100644
--- a/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/VertxRestTransport.java
+++ b/transports/transport-rest/transport-rest-vertx/src/main/java/org/apache/servicecomb/transport/rest/vertx/VertxRestTransport.java
@@ -35,7 +35,7 @@ import io.vertx.core.DeploymentOptions;
 
 @Component
 public class VertxRestTransport extends AbstractTransport {
-  private static final Logger log = LoggerFactory.getLogger(VertxRestTransport.class);
+  private static final Logger LOGGER = LoggerFactory.getLogger(VertxRestTransport.class);
 
   private RestTransportClient restClient;
 
@@ -59,7 +59,9 @@ public class VertxRestTransport extends AbstractTransport {
     }
 
     if (!NetUtils.canTcpListen(ep.getSocketAddress().getAddress(), ep.getPort())) {
-      log.info("can not listen {}, skip {}.", ep.getSocketAddress(), this.getClass().getName());
+      LOGGER.warn(
+          "Can not start VertxRestTransport, the port:{} may have been occupied. You can ignore this message if you are using a web container like tomcat.",
+          ep.getPort());
       return false;
     }
 

-- 
To stop receiving notification emails like this one, please contact
liubao@apache.org.