GEODE-2601: Updated based on feedback This closes #582
Project: http://git-wip-us.apache.org/repos/asf/geode/repo Commit: http://git-wip-us.apache.org/repos/asf/geode/commit/87c26be4 Tree: http://git-wip-us.apache.org/repos/asf/geode/tree/87c26be4 Diff: http://git-wip-us.apache.org/repos/asf/geode/diff/87c26be4 Branch: refs/heads/feature/GEODE-2804v3 Commit: 87c26be495de5a23290231e97826cb434277c66b Parents: 2d3e05e Author: YehEmily <emilyyeh1...@gmail.com> Authored: Fri Jun 16 09:06:56 2017 -0700 Committer: Ken Howe <kh...@pivotal.io> Committed: Mon Jun 19 10:05:12 2017 -0700 ---------------------------------------------------------------------- .../geode/distributed/internal/InternalDistributedSystem.java | 3 +-- .../org/apache/geode/distributed/internal/InternalLocator.java | 6 ++---- 2 files changed, 3 insertions(+), 6 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/geode/blob/87c26be4/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 4ed58af..1572355 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 @@ -2189,9 +2189,8 @@ public class InternalDistributedSystem extends DistributedSystem * @param resource the actual resource object. */ private void notifyResourceEventListeners(ResourceEvent event, Object resource) { - for (Iterator<ResourceEventsListener> iter = resourceListeners.iterator(); iter.hasNext();) { + for (ResourceEventsListener listener : resourceListeners) { try { - ResourceEventsListener listener = iter.next(); listener.handleEvent(event, resource); } catch (CancelException e) { // ignore http://git-wip-us.apache.org/repos/asf/geode/blob/87c26be4/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 3ff27ea..6eaaec2 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 @@ -469,15 +469,13 @@ public class InternalLocator extends Locator implements ConnectListener { // if security-log-file then create securityLogWriterAppender LogWriterAppenders.getOrCreateAppender(LogWriterAppenders.Identifier.SECURITY, true, false, this.config, false); - } - // 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, - false); + logWriter = LogWriterFactory.createLogWriterLogger(false, false, this.config, false); if (logger.isDebugEnabled()) { logger.debug("LogWriter for locator is created."); }