Skip to content

Commit f1ab27c

Browse files
author
Madelyn Kasula
authored
Merge pull request #378 from code-dot-org/use-logger-utils
Refactor Logger.getLogger references to use LoggerUtils
2 parents 4d4e029 + cab441e commit f1ab27c

File tree

3 files changed

+2
-12
lines changed

3 files changed

+2
-12
lines changed

org-code-javabuilder/lib/src/main/java/dev/javabuilder/WebSocketServer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ public void onOpen(Session session) {
133133

134134
@OnClose
135135
public void myOnClose() {
136-
Logger.getLogger(MAIN_LOGGER).info("WebSocket closed.");
136+
LoggerUtils.logInfo("WebSocket closed.");
137137
PerformanceTracker performanceTracker =
138138
(PerformanceTracker) JavabuilderContext.getInstance().get(PerformanceTracker.class);
139139
performanceTracker.logPerformance();

org-code-javabuilder/lib/src/main/java/org/code/javabuilder/PerformanceTracker.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
11
package org.code.javabuilder;
22

3-
import static org.code.protocol.LoggerNames.MAIN_LOGGER;
4-
53
import java.time.Clock;
64
import java.time.Instant;
7-
import java.util.logging.Logger;
85
import org.code.protocol.*;
96
import org.json.JSONObject;
107

@@ -103,6 +100,6 @@ public void logPerformance() {
103100
metricClient.publishCleanupTime(cleanupTime);
104101
}
105102

106-
Logger.getLogger(MAIN_LOGGER).info(logs.toString());
103+
LoggerUtils.logInfo(logs.toString());
107104
}
108105
}

org-code-javabuilder/validation/src/main/java/org/code/validation/support/NeighborhoodTracker.java

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,9 @@
11
package org.code.validation.support;
22

33
import static org.code.protocol.ClientMessageDetailKeys.*;
4-
import static org.code.protocol.LoggerNames.MAIN_LOGGER;
54

65
import java.util.*;
76
import java.util.function.Function;
8-
import java.util.logging.Logger;
97
import java.util.stream.Collectors;
108
import org.code.neighborhood.support.NeighborhoodSignalKey;
119
import org.code.neighborhood.support.World;
@@ -58,11 +56,6 @@ public void trackEvent(ClientMessage message) {
5856
}
5957

6058
if (!this.painterTrackers.containsKey(id) || !this.isInitialized) {
61-
Logger.getLogger(MAIN_LOGGER)
62-
.severe(
63-
String.format(
64-
"Error: received a painter event for an uninitialized painter: %s, %s\n",
65-
message.getValue(), message.getDetail()));
6659
return;
6760
}
6861

0 commit comments

Comments
 (0)