[
https://issues.apache.org/jira/browse/GEODE-2601?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16044780#comment-16044780
]
ASF GitHub Bot commented on GEODE-2601:
---------------------------------------
Github user kirklund commented on a diff in the pull request:
https://github.com/apache/geode/pull/571#discussion_r121188448
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/internal/InternalLocator.java
---
@@ -478,9 +478,8 @@ private InternalLocator(int port, File logF, File
stateF, InternalLogWriter logW
if (logWriter == null) {
logWriter = LogWriterFactory.createLogWriterLogger(false, false,
this.config,
!startDistributedSystem);
- if (logger.isDebugEnabled()) {
+ if (logger.isDebugEnabled())
--- End diff --
We've always had {'s in our style guidelines. So most of us are adding them
in for small if-blocks rather than removing them.
> Banner is logged twice during locator startup
> ---------------------------------------------
>
> Key: GEODE-2601
> URL: https://issues.apache.org/jira/browse/GEODE-2601
> Project: Geode
> Issue Type: Bug
> Components: logging
> Reporter: Patrick Rhomberg
> Assignee: Emily Yeh
> Priority: Minor
> Labels: LogBanner
>
> In locator log file, starting a locator in gfsh yields a log file containing
> "Licensed to the Apache [...]"
> First banner ends with:
> {noformat}[info 2017/03/06 14:29:29.995 PST loc1 <main> tid=0x1] Starting
> peer location for Distribution Locator on 10.118.33.239{noformat}
> Second banner ends with:
> {noformat}[info 2017/03/06 14:29:30.160 PST loc1 <main> tid=0x1] Starting
> membership services{noformat}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
