activemq-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From clebertsuco...@apache.org
Subject [1/2] activemq-artemis git commit: [ARTEMIS-1259] Log messages without prefixed id code in artemis-server-osgi
Date Tue, 27 Jun 2017 18:45:37 GMT
Repository: activemq-artemis
Updated Branches:
  refs/heads/master f79b42309 -> 70f6a2956


[ARTEMIS-1259] Log messages without prefixed id code in artemis-server-osgi


Project: http://git-wip-us.apache.org/repos/asf/activemq-artemis/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq-artemis/commit/028df470
Tree: http://git-wip-us.apache.org/repos/asf/activemq-artemis/tree/028df470
Diff: http://git-wip-us.apache.org/repos/asf/activemq-artemis/diff/028df470

Branch: refs/heads/master
Commit: 028df470309471842ca3cd9729533c4912107a84
Parents: f79b423
Author: Dmitrii Tikhomirov <dtikhomi@redhat.com>
Authored: Mon Jun 26 16:50:10 2017 +0200
Committer: Clebert Suconic <clebertsuconic@apache.org>
Committed: Tue Jun 27 14:36:24 2017 -0400

----------------------------------------------------------------------
 artemis-server-osgi/pom.xml                     |  8 ++-
 .../artemis/osgi/ActiveMQOsgiLogger.java        | 73 ++++++++++++++++++++
 .../artemis/osgi/DataSourceTracker.java         |  9 +--
 .../activemq/artemis/osgi/ProtocolTracker.java  | 14 ++--
 4 files changed, 87 insertions(+), 17 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/028df470/artemis-server-osgi/pom.xml
----------------------------------------------------------------------
diff --git a/artemis-server-osgi/pom.xml b/artemis-server-osgi/pom.xml
index 185fa9e..21306cd 100644
--- a/artemis-server-osgi/pom.xml
+++ b/artemis-server-osgi/pom.xml
@@ -84,8 +84,6 @@
          <scope>provided</scope>
          <optional>true</optional>
       </dependency>
-
-
       <dependency>
          <groupId>org.jboss.logmanager</groupId>
          <artifactId>jboss-logmanager</artifactId>
@@ -93,6 +91,12 @@
          <scope>provided</scope>
       </dependency>
       <dependency>
+         <groupId>org.jboss.logging</groupId>
+         <artifactId>jboss-logging-processor</artifactId>
+         <scope>provided</scope>
+         <optional>true</optional>
+      </dependency>
+      <dependency>
          <groupId>xalan</groupId>
          <artifactId>xalan</artifactId>
          <version>2.7.2</version>

http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/028df470/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/ActiveMQOsgiLogger.java
----------------------------------------------------------------------
diff --git a/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/ActiveMQOsgiLogger.java
b/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/ActiveMQOsgiLogger.java
new file mode 100644
index 0000000..c51373d
--- /dev/null
+++ b/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/ActiveMQOsgiLogger.java
@@ -0,0 +1,73 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ *    http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.activemq.artemis.osgi;
+
+import org.jboss.logging.BasicLogger;
+import org.jboss.logging.Logger;
+import org.jboss.logging.annotations.Cause;
+import org.jboss.logging.annotations.LogMessage;
+import org.jboss.logging.annotations.Message;
+import org.jboss.logging.annotations.MessageLogger;
+
+/**
+ * Logger code 58
+ *
+ * each message id must be 6 digits long starting with 57, the 3rd digit donates the level
so
+ *
+ * INF0  1
+ * WARN  2
+ * DEBUG 3
+ * ERROR 4
+ * TRACE 5
+ * FATAL 6
+ *
+ * so an INFO message would be 581000 to 581999
+ */
+@MessageLogger(projectCode = "AMQ")
+public interface ActiveMQOsgiLogger extends BasicLogger {
+
+   /**
+   * * The default logger.
+   */
+   ActiveMQOsgiLogger LOGGER = Logger.getMessageLogger(ActiveMQOsgiLogger.class, ActiveMQOsgiLogger.class.getPackage().getName());
+
+   @LogMessage(level = Logger.Level.INFO)
+   @Message(id = 581000, value = "Broker config {0} found. Tracking protocols {1}", format
= Message.Format.MESSAGE_FORMAT)
+   void brokerConfigFound(String name, String protocols);
+
+   @LogMessage(level = Logger.Level.INFO)
+   @Message(id = 581001, value = "Required protocol {0} was added for broker {1}. {2}", format
= Message.Format.MESSAGE_FORMAT)
+   void protocolWasAddedForBroker(String protocol, String name, String message);
+
+   @LogMessage(level = Logger.Level.INFO)
+   @Message(id = 581002, value = "Required protocol {0} was removed for broker {1}. {2}",
format = Message.Format.MESSAGE_FORMAT)
+   void protocolWasRemovedForBroker(String protocol, String name, String message);
+
+   @LogMessage(level = Logger.Level.WARN)
+   @Message(id = 582000, value = "Error starting broker: {0}", format = Message.Format.MESSAGE_FORMAT)
+   void errorStartingBroker(@Cause Exception e, String name);
+
+   @LogMessage(level = Logger.Level.WARN)
+   @Message(id = 582001, value = "Error stopping broker: {0}", format = Message.Format.MESSAGE_FORMAT)
+   void errorStoppingBroker(@Cause Exception e, String name);
+
+   @LogMessage(level = Logger.Level.WARN)
+   @Message(id = 582002, value = "Error getting dataSource provider infos.", format = Message.Format.MESSAGE_FORMAT)
+   void errorGettingDataSourceProviderInfo(@Cause Exception e);
+
+}
+

http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/028df470/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/DataSourceTracker.java
----------------------------------------------------------------------
diff --git a/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/DataSourceTracker.java
b/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/DataSourceTracker.java
index 941d39f..bfc7362 100644
--- a/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/DataSourceTracker.java
+++ b/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/DataSourceTracker.java
@@ -19,8 +19,6 @@ package org.apache.activemq.artemis.osgi;
 import javax.sql.DataSource;
 import java.sql.Connection;
 import java.sql.SQLException;
-import java.util.logging.Level;
-import java.util.logging.Logger;
 
 import org.apache.activemq.artemis.core.config.storage.DatabaseStorageConfiguration;
 import org.apache.activemq.artemis.jdbc.store.drivers.JDBCUtils;
@@ -30,7 +28,6 @@ import org.osgi.util.tracker.ServiceTrackerCustomizer;
 
 public class DataSourceTracker implements ServiceTrackerCustomizer<DataSource, DataSource>
{
 
-   private static final Logger LOG = Logger.getLogger(ProtocolTracker.class.getName());
    private final String name;
    private final BundleContext context;
    private final DatabaseStorageConfiguration dsc;
@@ -51,13 +48,13 @@ public class DataSourceTracker implements ServiceTrackerCustomizer<DataSource,
D
       try (Connection conn = dataSource.getConnection()) {
          dsc.setSqlProvider(JDBCUtils.getSQLProviderFactory(conn.getMetaData().getURL()));
       } catch (SQLException ex) {
-         LOG.log(Level.WARNING, "Error getting dataSource provider infos", ex);
+         ActiveMQOsgiLogger.LOGGER.errorGettingDataSourceProviderInfo(ex);
       }
       callback.setDataSourceDependency(false);
       try {
          callback.start();
       } catch (Exception ex) {
-         LOG.log(Level.WARNING, "Error starting broker " + name, ex);
+         ActiveMQOsgiLogger.LOGGER.errorStartingBroker(ex, name);
       }
       return dataSource;
    }
@@ -73,7 +70,7 @@ public class DataSourceTracker implements ServiceTrackerCustomizer<DataSource,
D
       try {
          callback.stop();
       } catch (Exception ex) {
-         LOG.log(Level.WARNING, "Error stopping broker " + name, ex);
+         ActiveMQOsgiLogger.LOGGER.errorStoppingBroker(ex, name);
       }
    }
 }

http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/028df470/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/ProtocolTracker.java
----------------------------------------------------------------------
diff --git a/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/ProtocolTracker.java
b/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/ProtocolTracker.java
index e01f629..5ab2d3a 100644
--- a/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/ProtocolTracker.java
+++ b/artemis-server-osgi/src/main/java/org/apache/activemq/artemis/osgi/ProtocolTracker.java
@@ -21,8 +21,6 @@ import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.logging.Level;
-import java.util.logging.Logger;
 
 import org.apache.activemq.artemis.api.core.Interceptor;
 import org.apache.activemq.artemis.spi.core.protocol.ProtocolManagerFactory;
@@ -39,7 +37,6 @@ import org.osgi.util.tracker.ServiceTrackerCustomizer;
 @SuppressWarnings("rawtypes")
 public class ProtocolTracker implements ServiceTrackerCustomizer<ProtocolManagerFactory<Interceptor>,
ProtocolManagerFactory<Interceptor>> {
 
-   private static Logger LOG = Logger.getLogger(ProtocolTracker.class.getName());
    private String name;
    private BundleContext context;
    private Map<String, Boolean> protocols;
@@ -56,7 +53,7 @@ public class ProtocolTracker implements ServiceTrackerCustomizer<ProtocolManager
       for (String requiredProtocol : requiredProtocols) {
          this.protocols.put(requiredProtocol, false);
       }
-      LOG.info("Broker config " + name + " found. Tracking protocols " + Arrays.asList(requiredProtocols));
+      ActiveMQOsgiLogger.LOGGER.brokerConfigFound(name, Arrays.asList(requiredProtocols).toString());
    }
 
    @Override
@@ -90,13 +87,12 @@ public class ProtocolTracker implements ServiceTrackerCustomizer<ProtocolManager
       if (present != null && !present) {
          this.protocols.put(protocol, true);
          List<String> missing = getMissing();
-         LOG.info("Required protocol " + protocol + " was added for broker " + name + ".
" +
-                     (missing.isEmpty() ? "Starting broker." : "Still waiting for " + missing));
+         ActiveMQOsgiLogger.LOGGER.protocolWasAddedForBroker(protocol, name, (missing.isEmpty()
? "Starting broker." : "Still waiting for " + missing));
          if (missing.isEmpty()) {
             try {
                callback.start();
             } catch (Exception e) {
-               LOG.log(Level.WARNING, "Error starting broker " + name, e);
+               ActiveMQOsgiLogger.LOGGER.errorStartingBroker(e, name);
             }
          }
       }
@@ -106,12 +102,12 @@ public class ProtocolTracker implements ServiceTrackerCustomizer<ProtocolManager
       Boolean present = this.protocols.get(protocol);
       if (present != null && present) {
          List<String> missing = getMissing();
-         LOG.info("Required protocol " + protocol + " was removed for broker " + name + ".
" + (missing.isEmpty() ? "Stopping broker. " : ""));
+         ActiveMQOsgiLogger.LOGGER.protocolWasRemovedForBroker(protocol, name, (missing.isEmpty()
? "Stopping broker. " : ""));
          if (missing.isEmpty()) {
             try {
                callback.stop();
             } catch (Exception e) {
-               LOG.log(Level.WARNING, "Error stopping broker " + name, e);
+               ActiveMQOsgiLogger.LOGGER.errorStoppingBroker(e, name);
             }
          }
          this.protocols.put(protocol, false);


Mime
View raw message