Skip to content

Commit

Permalink
GH-1970: fix logger name in SPARQLProtocolSession
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
aschwarte10 authored and abrokenjester committed Mar 6, 2020
1 parent c212188 commit 43ef209
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -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 *
Expand Down

0 comments on commit 43ef209

Please sign in to comment.