You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Marko Donke (JIRA)" <ji...@apache.org> on 2018/08/08 10:04:00 UTC
[jira] [Commented] (SLING-7583) Log tracer runs into
ClassCastException for oak-queries
[ https://issues.apache.org/jira/browse/SLING-7583?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16572975#comment-16572975 ]
Marko Donke commented on SLING-7583:
------------------------------------
Is there a way for a convenient download of Log Tracer 1.0.8 ?
Can't find it here: https://sling.apache.org/downloads.cgi
> Log tracer runs into ClassCastException for oak-queries
> -------------------------------------------------------
>
> Key: SLING-7583
> URL: https://issues.apache.org/jira/browse/SLING-7583
> Project: Sling
> Issue Type: Bug
> Components: Extensions
> Affects Versions: Log Tracer 1.0.6, Log Tracer 1.0.4
> Reporter: Dirk Rudolph
> Assignee: Chetan Mehrotra
> Priority: Minor
> Fix For: Log Tracer 1.0.8
>
>
> Oak's {{QueryEngineImpl}} logs the following "cost: {} for query {}" with the first parameter being a primitive {{double}} and the second being an instance of {{QueryImpl}} [1]. On the other hand the TraceContext casts thats second parameter to {{String}} without a type check [2]. I would propose to use {{String.valueOf()}} here to get the proper {{toString()}} implementation of the {{QueryImpl}} object.
>
> [1] [https://github.com/apache/jackrabbit-oak/blob/jackrabbit-oak-1.6.7/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryEngineImpl.java#L311]
> [2] [https://github.com/apache/sling-org-apache-sling-tracer/blob/master/src/main/java/org/apache/sling/tracer/internal/TracerContext.java#L92]
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)