sling-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chetan Mehrotra (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SLING-3049) Make Logback Stacktrace Packaging data support OSGi aware
Date Thu, 02 Nov 2017 09:48:00 GMT

    [ https://issues.apache.org/jira/browse/SLING-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16235466#comment-16235466
] 

Chetan Mehrotra commented on SLING-3049:
----------------------------------------

Thanks [~karlpauls] for the feedback

bq. Obviously, it suffers a little from not being able to get to the real classes - i.e.,
it will not report on classes that are provided from more than one bundle

Yes. If same package is loaded by multiple bundles then this impl would not provide any info.
But in most cases the packages are unique so should be ok for Sling like setup

bq.  I guess the only question I would have is if this could be problematic for tools/scripts/others
that rely on a certain layout of a stacktrace. Not sure that is important. 

I mostly use Intellij Stacktrace Analyzer and it is able to work with that

bq. I suppose you could use some bytecode magic to weave the information about the bundle
source into the "Source" field of the class and parse it out later when you need it but that
probably isn't a good idea

That would be really cool and nifty use of weaving hook!. But for some other day :)

> Make Logback Stacktrace Packaging data support OSGi aware
> ---------------------------------------------------------
>
>                 Key: SLING-3049
>                 URL: https://issues.apache.org/jira/browse/SLING-3049
>             Project: Sling
>          Issue Type: Improvement
>          Components: Commons
>            Reporter: Chetan Mehrotra
>            Assignee: Chetan Mehrotra
>            Priority: Major
>              Labels: logback
>         Attachments: SLING-3049.patch, buildbot-exceptions-while-stopping-jetty.txt
>
>
> Logback provides a useful feature where it dumps the Class packaging Data along with
the stacktrace [1]. This provides a quick view of the location from where classes in a given
stacktrace are coming. Its default logic does not work properly in OSGi env. Hence it would
be useful to patch its logic to become OSGi aware
> [1] http://logback.qos.ch/reasonsToSwitch.html#packagingData



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message