Skip to content

Commit 2f5c6b7

Browse files
committed
publisher: Switch to more efficient slf4j logging
- pom: Add JavaDoc link for buabook-common
1 parent f089b9e commit 2f5c6b7

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

pom.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,10 @@
8989
<show>private</show>
9090
<author>true</author>
9191
<nohelp>true</nohelp>
92+
93+
<links>
94+
<link>https://buabook.github.io/javadoc/buabook-common/</link>
95+
</links>
9296
</configuration>
9397
<executions>
9498
<execution>

src/main/java/com/buabook/kdb/publisher/KdbPublisherManager.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ public KdbPublisherManager() {
3838

3939
public void addPublisher(KdbProcess server) throws KdbTargetProcessUnavailableException, KdbPublisherAlreadyExistsException {
4040
if(publishers.containsKey(server)) {
41-
log.error("This manager already contains a publisher to this KDB process! [ Process: " + server.toString() + " ]");
41+
log.error("This manager already contains a publisher to this KDB process! [ Process: {} ]", server);
4242
throw new KdbPublisherAlreadyExistsException(server.toString());
4343
}
4444

@@ -80,11 +80,11 @@ public List<KdbProcess> getTargetProcesses() {
8080
*/
8181
public void disconnect(KdbProcess server) throws KdbPublisherDoesNotExistException {
8282
if(! publishers.containsKey(server)) {
83-
log.error("The specified KDB process does not exist within this manager [ Process: " + server.toString() + " ]");
83+
log.error("The specified KDB process does not exist within this manager [ Process: {} ]", server);
8484
throw new KdbPublisherDoesNotExistException(server.toString());
8585
}
8686

87-
log.info("Disconnect request received for server: " + server.toString());
87+
log.info("Disconnect request received for server: {}", server);
8888

8989
publishers.get(server).disconnect();
9090
publishers.remove(server);

0 commit comments

Comments
 (0)