Github user xristy commented on a diff in the pull request:
https://github.com/apache/jena/pull/319#discussion_r153838075
--- Diff:
jena-text/src/main/java/org/apache/jena/query/text/TextQueryPF.java ---
@@ -224,12 +234,16 @@ private QueryIterator resultsToQueryIterator(Binding
binding, Node s, Node score
}
private QueryIterator variableSubject(Binding binding, Node s, Node
score, Node literal, StrMatch match, ExecutionContext execCxt) {
+ if (log.isTraceEnabled())
+ log.trace("variableSubject: {}", match) ;
--- End diff --
I'm happy to amend by removing the unnecessary `log.isTraceEnabled()`. I
was intending to follow the coding practices elsewhere in TextQueryPF and
TextIndexLucene, should I amend the unneeded occurrences of
`log.isDebugEnabled()` as well?
---
