Short logger names to reduce visual clutter in log output #322
@@ -99,7 +99,7 @@ public class ServerApp {
|
|||||||
public static void start(String arg) {
|
public static void start(String arg) {
|
||||||
int port = Integer.parseInt(arg);
|
int port = Integer.parseInt(arg);
|
||||||
|
|
||||||
Logger logger = LogManager.getLogger(ServerApp.class);
|
Logger logger = LogManager.getLogger(ServerApp.class.getSimpleName());
|
||||||
logger.info("Starting server at port {}", port);
|
logger.info("Starting server at port {}", port);
|
||||||
|
|
||||||
EventBus eventBus = new EventBus();
|
EventBus eventBus = new EventBus();
|
||||||
|
|||||||
+2
-1
@@ -23,7 +23,8 @@ import org.apache.logging.log4j.Logger;
|
|||||||
public class JoinLobbyHandler extends CommandHandler<JoinLobbyRequest> {
|
public class JoinLobbyHandler extends CommandHandler<JoinLobbyRequest> {
|
||||||
private final LobbyManager lobbyManager;
|
private final LobbyManager lobbyManager;
|
||||||
private final UserRegistry userRegistry;
|
private final UserRegistry userRegistry;
|
||||||
private static final Logger LOGGER = LogManager.getLogger(JoinLobbyHandler.class);
|
private static final Logger LOGGER =
|
||||||
|
LogManager.getLogger(JoinLobbyHandler.class.getSimpleName());
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Create a new {@link JoinLobbyHandler}.
|
* Create a new {@link JoinLobbyHandler}.
|
||||||
|
|||||||
+2
-1
@@ -14,7 +14,8 @@ import org.apache.logging.log4j.Logger;
|
|||||||
/** Periodically removes expired empty lobbies and notifies connected sessions. */
|
/** Periodically removes expired empty lobbies and notifies connected sessions. */
|
||||||
public class LobbyCleanupJob implements Runnable {
|
public class LobbyCleanupJob implements Runnable {
|
||||||
|
|
||||||
private static final Logger LOGGER = LogManager.getLogger(LobbyCleanupJob.class);
|
private static final Logger LOGGER =
|
||||||
|
LogManager.getLogger(LobbyCleanupJob.class.getSimpleName());
|
||||||
|
|
||||||
private final LobbyManager lobbyManager;
|
private final LobbyManager lobbyManager;
|
||||||
private final SessionManager sessionManager;
|
private final SessionManager sessionManager;
|
||||||
|
|||||||
+1
-1
@@ -15,7 +15,7 @@ public class UserCleanupJob implements Runnable {
|
|||||||
private final Duration reconnectThreshold;
|
private final Duration reconnectThreshold;
|
||||||
|
|
||||||
public UserCleanupJob(UserRegistry registry, Duration reconnectThreshold) {
|
public UserCleanupJob(UserRegistry registry, Duration reconnectThreshold) {
|
||||||
this.logger = LogManager.getLogger(UserCleanupJob.class);
|
this.logger = LogManager.getLogger(UserCleanupJob.class.getSimpleName());
|
||||||
this.registry = registry;
|
this.registry = registry;
|
||||||
this.reconnectThreshold = reconnectThreshold;
|
this.reconnectThreshold = reconnectThreshold;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -9,7 +9,7 @@ import org.apache.logging.log4j.Logger;
|
|||||||
public class UserFactory {
|
public class UserFactory {
|
||||||
private final UserRegistry registry;
|
private final UserRegistry registry;
|
||||||
private final AtomicInteger anonymousCounter = new AtomicInteger(1);
|
private final AtomicInteger anonymousCounter = new AtomicInteger(1);
|
||||||
private static final Logger LOGGER = LogManager.getLogger(UserFactory.class);
|
private static final Logger LOGGER = LogManager.getLogger(UserFactory.class.getSimpleName());
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Creates a new UserFactory backed by the given registry.
|
* Creates a new UserFactory backed by the given registry.
|
||||||
|
|||||||
@@ -26,7 +26,7 @@ public class NetworkManager implements Runnable {
|
|||||||
*/
|
*/
|
||||||
public NetworkManager(Integer port, SessionManager sessionManager, CommandRouter router) {
|
public NetworkManager(Integer port, SessionManager sessionManager, CommandRouter router) {
|
||||||
this.port = port;
|
this.port = port;
|
||||||
this.logger = LogManager.getLogger(NetworkManager.class);
|
this.logger = LogManager.getLogger(NetworkManager.class.getSimpleName());
|
||||||
this.thread = new Thread(this, "networkManager");
|
this.thread = new Thread(this, "networkManager");
|
||||||
this.running = true;
|
this.running = true;
|
||||||
this.sessionManager = sessionManager;
|
this.sessionManager = sessionManager;
|
||||||
|
|||||||
+1
-1
@@ -15,7 +15,7 @@ public class SessionDisconnectJob implements Runnable {
|
|||||||
|
|
||||||
public SessionDisconnectJob(
|
public SessionDisconnectJob(
|
||||||
SessionManager sessionManager, EventBus eventBus, Duration timeoutThreshold) {
|
SessionManager sessionManager, EventBus eventBus, Duration timeoutThreshold) {
|
||||||
this.logger = LogManager.getLogger(SessionDisconnectJob.class);
|
this.logger = LogManager.getLogger(SessionDisconnectJob.class.getSimpleName());
|
||||||
this.sessionManager = sessionManager;
|
this.sessionManager = sessionManager;
|
||||||
this.eventBus = eventBus;
|
this.eventBus = eventBus;
|
||||||
this.timeoutThreshold = timeoutThreshold;
|
this.timeoutThreshold = timeoutThreshold;
|
||||||
|
|||||||
+1
-1
@@ -24,7 +24,7 @@ public class SessionManager {
|
|||||||
public SessionManager(EventBus eventBus, CommandParserDispatcher dispatcher) {
|
public SessionManager(EventBus eventBus, CommandParserDispatcher dispatcher) {
|
||||||
this.sessions = new ConcurrentHashMap<>();
|
this.sessions = new ConcurrentHashMap<>();
|
||||||
this.eventBus = eventBus;
|
this.eventBus = eventBus;
|
||||||
this.logger = LogManager.getLogger(SessionManager.class);
|
this.logger = LogManager.getLogger(SessionManager.class.getSimpleName());
|
||||||
this.dispatcher = dispatcher;
|
this.dispatcher = dispatcher;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
+1
-1
@@ -41,7 +41,7 @@ public class SessionReader implements Runnable {
|
|||||||
this.router = session.getRouter();
|
this.router = session.getRouter();
|
||||||
this.logger =
|
this.logger =
|
||||||
LogManager.getLogger(
|
LogManager.getLogger(
|
||||||
SessionReader.class.toString() + "-" + session.getId().value());
|
SessionReader.class.getSimpleName() + "-" + session.getId().value());
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
|||||||
+1
-1
@@ -18,7 +18,7 @@ public class SessionWriter implements Runnable {
|
|||||||
this.queue = session.getResponseQueue();
|
this.queue = session.getResponseQueue();
|
||||||
this.logger =
|
this.logger =
|
||||||
LogManager.getLogger(
|
LogManager.getLogger(
|
||||||
SessionReader.class.toString() + "-" + session.getId().value());
|
SessionReader.class.getSimpleName() + "-" + session.getId().value());
|
||||||
}
|
}
|
||||||
|
|
||||||
public void run() {
|
public void run() {
|
||||||
|
|||||||
Reference in New Issue
Block a user