Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 89426200CDA for ; Tue, 27 Jun 2017 07:00:23 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 880F9160BFD; Tue, 27 Jun 2017 05:00:23 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 89D5D160C01 for ; Tue, 27 Jun 2017 07:00:22 +0200 (CEST) Received: (qmail 39212 invoked by uid 500); 27 Jun 2017 05:00:21 -0000 Mailing-List: contact commits-help@geode.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@geode.apache.org Delivered-To: mailing list commits@geode.apache.org Received: (qmail 38044 invoked by uid 99); 27 Jun 2017 05:00:19 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 27 Jun 2017 05:00:19 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id BACB3F3238; Tue, 27 Jun 2017 05:00:16 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: zhouxj@apache.org To: commits@geode.apache.org Date: Tue, 27 Jun 2017 05:00:22 -0000 Message-Id: <7fd187d2520e44b5a63d6b0b43917c1d@git.apache.org> In-Reply-To: References: X-Mailer: ASF-Git Admin Mailer Subject: [07/46] geode git commit: GEODE-2601: Fixing banner being logged twice during locator startup. archived-at: Tue, 27 Jun 2017 05:00:23 -0000 GEODE-2601: Fixing banner being logged twice during locator startup. Project: http://git-wip-us.apache.org/repos/asf/geode/repo Commit: http://git-wip-us.apache.org/repos/asf/geode/commit/2d3e05ec Tree: http://git-wip-us.apache.org/repos/asf/geode/tree/2d3e05ec Diff: http://git-wip-us.apache.org/repos/asf/geode/diff/2d3e05ec Branch: refs/heads/feature/GEM-1483 Commit: 2d3e05ecedf724af8d413f50567afc7f9a87e343 Parents: 3154424 Author: YehEmily Authored: Mon Jun 12 13:55:24 2017 -0700 Committer: Ken Howe Committed: Mon Jun 19 09:52:13 2017 -0700 ---------------------------------------------------------------------- .../internal/InternalDistributedSystem.java | 24 ++++++++++++ .../distributed/internal/InternalLocator.java | 41 ++++++++++---------- .../internal/logging/LogWriterFactory.java | 20 ++++------ 3 files changed, 53 insertions(+), 32 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/geode/blob/2d3e05ec/geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java ---------------------------------------------------------------------- diff --git a/geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java b/geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java index d77e8b9..4ed58af 100644 --- a/geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java +++ b/geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java @@ -18,6 +18,30 @@ package org.apache.geode.distributed.internal; import static org.apache.geode.distributed.ConfigurationProperties.LOCATORS; import static org.apache.geode.distributed.ConfigurationProperties.MCAST_PORT; +import java.io.File; +import java.io.IOException; +import java.io.Reader; +import java.lang.reflect.Array; +import java.net.InetAddress; +import java.util.ArrayList; +import java.util.Date; +import java.util.HashSet; +import java.util.Iterator; +import java.util.LinkedHashSet; +import java.util.List; +import java.util.Properties; +import java.util.Set; +import java.util.SortedSet; +import java.util.StringTokenizer; +import java.util.TreeSet; +import java.util.concurrent.ConcurrentHashMap; +import java.util.concurrent.CopyOnWriteArrayList; +import java.util.concurrent.TimeUnit; +import java.util.concurrent.atomic.AtomicLong; +import java.util.concurrent.atomic.AtomicReference; + +import org.apache.logging.log4j.Logger; + import org.apache.geode.CancelCriterion; import org.apache.geode.CancelException; import org.apache.geode.ForcedDisconnectException; http://git-wip-us.apache.org/repos/asf/geode/blob/2d3e05ec/geode-core/src/main/java/org/apache/geode/distributed/internal/InternalLocator.java ---------------------------------------------------------------------- diff --git a/geode-core/src/main/java/org/apache/geode/distributed/internal/InternalLocator.java b/geode-core/src/main/java/org/apache/geode/distributed/internal/InternalLocator.java index ae548c4..3ff27ea 100644 --- a/geode-core/src/main/java/org/apache/geode/distributed/internal/InternalLocator.java +++ b/geode-core/src/main/java/org/apache/geode/distributed/internal/InternalLocator.java @@ -18,7 +18,24 @@ import static org.apache.geode.distributed.ConfigurationProperties.BIND_ADDRESS; import static org.apache.geode.distributed.ConfigurationProperties.CACHE_XML_FILE; import static org.apache.geode.distributed.ConfigurationProperties.LOCATORS; +import java.io.File; +import java.io.IOException; +import java.net.ConnectException; +import java.net.InetAddress; +import java.net.UnknownHostException; +import java.util.Collection; +import java.util.HashMap; +import java.util.HashSet; +import java.util.Properties; +import java.util.concurrent.Callable; +import java.util.concurrent.ExecutorService; +import java.util.concurrent.Future; +import java.util.concurrent.TimeUnit; +import java.util.concurrent.atomic.AtomicBoolean; + import org.apache.commons.lang.StringUtils; +import org.apache.logging.log4j.Logger; + import org.apache.geode.CancelException; import org.apache.geode.cache.CacheFactory; import org.apache.geode.cache.GemFireCache; @@ -65,22 +82,6 @@ import org.apache.geode.management.internal.configuration.handlers.SharedConfigu import org.apache.geode.management.internal.configuration.messages.ConfigurationRequest; import org.apache.geode.management.internal.configuration.messages.SharedConfigurationStatusRequest; import org.apache.geode.management.internal.configuration.messages.SharedConfigurationStatusResponse; -import org.apache.logging.log4j.Logger; - -import java.io.File; -import java.io.IOException; -import java.net.ConnectException; -import java.net.InetAddress; -import java.net.UnknownHostException; -import java.util.Collection; -import java.util.HashMap; -import java.util.HashSet; -import java.util.Properties; -import java.util.concurrent.Callable; -import java.util.concurrent.ExecutorService; -import java.util.concurrent.Future; -import java.util.concurrent.TimeUnit; -import java.util.concurrent.atomic.AtomicBoolean; /** * Provides the implementation of a distribution {@code Locator} as well as internal-only @@ -469,17 +470,17 @@ public class InternalLocator extends Locator implements ConnectListener { LogWriterAppenders.getOrCreateAppender(LogWriterAppenders.Identifier.SECURITY, true, false, this.config, false); - } else { - // do not create a LogWriterAppender for security -- let it go through to logWriterAppender } + // do not create a LogWriterAppender for security -- let it go through to logWriterAppender } // LOG: create LogWriters for GemFireTracer (or use whatever was passed in) if (logWriter == null) { logWriter = LogWriterFactory.createLogWriterLogger(false, false, this.config, - !startDistributedSystem); - if (logger.isDebugEnabled()) + false); + if (logger.isDebugEnabled()) { logger.debug("LogWriter for locator is created."); + } } if (securityLogWriter == null) { http://git-wip-us.apache.org/repos/asf/geode/blob/2d3e05ec/geode-core/src/main/java/org/apache/geode/internal/logging/LogWriterFactory.java ---------------------------------------------------------------------- diff --git a/geode-core/src/main/java/org/apache/geode/internal/logging/LogWriterFactory.java b/geode-core/src/main/java/org/apache/geode/internal/logging/LogWriterFactory.java index c4fecfd..4202afd 100755 --- a/geode-core/src/main/java/org/apache/geode/internal/logging/LogWriterFactory.java +++ b/geode-core/src/main/java/org/apache/geode/internal/logging/LogWriterFactory.java @@ -14,7 +14,7 @@ */ package org.apache.geode.internal.logging; -import static org.apache.geode.distributed.ConfigurationProperties.*; +import static org.apache.geode.distributed.ConfigurationProperties.LOG_LEVEL; import org.apache.geode.distributed.internal.DistributionConfig; import org.apache.geode.distributed.internal.InternalDistributedSystem; @@ -79,16 +79,13 @@ public class LogWriterFactory { } // log the banner - if (!Boolean.getBoolean(InternalLocator.INHIBIT_DM_BANNER)) { - if (InternalDistributedSystem.getReconnectAttemptCounter() == 0 // avoid filling up logs - // during auto-reconnect - && !isSecure // && !isLoner /* do this on a loner to fix bug 35602 */ - ) { - // LOG:CONFIG: - logger.info(LogMarker.CONFIG, Banner.getString(null)); - } - System.setProperty(InternalLocator.INHIBIT_DM_BANNER, "true"); // Ensure no more banners will - // be logged + if (!Boolean.getBoolean(InternalLocator.INHIBIT_DM_BANNER) + && InternalDistributedSystem.getReconnectAttemptCounter() == 0 // avoid filling up logs + // during auto-reconnect + && !isSecure // && !isLoner /* do this on a loner to fix bug 35602 */ + && logConfig) { + // LOG:CONFIG: + logger.info(LogMarker.CONFIG, Banner.getString(null)); } else { logger.debug("skipping banner - " + InternalLocator.INHIBIT_DM_BANNER + " is set to true"); } @@ -101,7 +98,6 @@ public class LogWriterFactory { LocalizedStrings.InternalDistributedSystem_STARTUP_CONFIGURATIONN_0, config.toLoggerString())); } - return logger; } }