From 43ef20986009fa55276cb93d76a540946bb409a5 Mon Sep 17 00:00:00 2001 From: Andreas Schwarte Date: Thu, 5 Mar 2020 07:30:53 +0100 Subject: [PATCH] GH-1970: fix logger name in SPARQLProtocolSession The SparqlProtocolSession uses this.getClass() as a name for the logger. When having an extension of the class, this leads to misleading log output. Particularly the logging levels are also harder to control (w.r.t namespacing and packaging). The general recommendation for logger names should be to use the class name, at least it should adhere to the rdf4j package structure. --- .../org/eclipse/rdf4j/http/client/SPARQLProtocolSession.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/http/client/src/main/java/org/eclipse/rdf4j/http/client/SPARQLProtocolSession.java b/core/http/client/src/main/java/org/eclipse/rdf4j/http/client/SPARQLProtocolSession.java index 83c65015d75..c109458c24c 100644 --- a/core/http/client/src/main/java/org/eclipse/rdf4j/http/client/SPARQLProtocolSession.java +++ b/core/http/client/src/main/java/org/eclipse/rdf4j/http/client/SPARQLProtocolSession.java @@ -148,7 +148,7 @@ public class SPARQLProtocolSession implements HttpClientDependent, AutoCloseable */ private final int maximumUrlLength; - final Logger logger = LoggerFactory.getLogger(this.getClass()); + final Logger logger = LoggerFactory.getLogger(SPARQLProtocolSession.class); /*-----------* * Variables *