From 10bac0ad21fca0c5ac8bf6d7dc712c5899c2d0e6 Mon Sep 17 00:00:00 2001
From: Nicolas Capponi <nicolas.capponi@forgerock.com>
Date: Thu, 06 Feb 2014 13:51:13 +0000
Subject: [PATCH] Fix OPENDJ-1289 Runtime debug configuration changes have no effect
---
opendj3-server-dev/src/server/org/opends/server/loggers/ErrorLogger.java | 62 +++++++++++++-----------------
1 files changed, 27 insertions(+), 35 deletions(-)
diff --git a/opendj3-server-dev/src/server/org/opends/server/loggers/ErrorLogger.java b/opendj3-server-dev/src/server/org/opends/server/loggers/ErrorLogger.java
index cc99896..c9d104d 100644
--- a/opendj3-server-dev/src/server/org/opends/server/loggers/ErrorLogger.java
+++ b/opendj3-server-dev/src/server/org/opends/server/loggers/ErrorLogger.java
@@ -27,6 +27,7 @@
package org.opends.server.loggers;
import static org.opends.messages.ConfigMessages.*;
+import java.util.Collection;
import org.forgerock.i18n.LocalizableMessage;
import org.opends.messages.Severity;
import org.opends.server.admin.ClassPropertyDefinition;
@@ -66,7 +67,7 @@
/**
* The constructor for this class.
*/
- public ErrorLogger()
+ private ErrorLogger()
{
super((Class) ErrorLogPublisher.class,
ERR_CONFIG_LOGGER_INVALID_ERROR_LOGGER_CLASS);
@@ -82,41 +83,9 @@
/** {@inheritDoc} */
@Override
- protected LoggerStorage<ErrorLogPublisher<ErrorLogPublisherCfg>,
- ErrorLogPublisherCfg> getStorage()
+ protected Collection<ErrorLogPublisher<ErrorLogPublisherCfg>> getLogPublishers()
{
- return loggerStorage;
- }
-
- /**
- * Add an error log publisher to the error logger.
- *
- * @param publisher The error log publisher to add.
- */
- public synchronized static void addErrorLogPublisher(
- ErrorLogPublisher publisher)
- {
- loggerStorage.addLogPublisher(publisher);
- }
-
- /**
- * Remove an error log publisher from the error logger.
- *
- * @param publisher The error log publisher to remove.
- * @return True if the error log publisher is removed or false otherwise.
- */
- public synchronized static boolean removeErrorLogPublisher(
- ErrorLogPublisher publisher)
- {
- return loggerStorage.removeLogPublisher(publisher);
- }
-
- /**
- * Removes all existing error log publishers from the logger.
- */
- public synchronized static void removeAllErrorLogPublishers()
- {
- loggerStorage.removeAllLogPublishers();
+ return loggerStorage.getLogPublishers();
}
/**
@@ -188,4 +157,27 @@
return false;
}
+ /** {@inheritDoc} */
+ @Override
+ public final synchronized void addLogPublisher(
+ ErrorLogPublisher<ErrorLogPublisherCfg> publisher)
+ {
+ loggerStorage.addLogPublisher(publisher);
+ }
+
+ /** {@inheritDoc} */
+ @Override
+ public final synchronized boolean removeLogPublisher(
+ ErrorLogPublisher<ErrorLogPublisherCfg> publisher)
+ {
+ return loggerStorage.removeLogPublisher(publisher);
+ }
+
+ /** {@inheritDoc} */
+ @Override
+ public final synchronized void removeAllLogPublishers()
+ {
+ loggerStorage.removeAllLogPublishers();
+ }
+
}
--
Gitblit v1.10.0