[
https://issues.apache.org/jira/browse/LOG4J2-2899?focusedWorklogId=460775&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-460775
]
ASF GitHub Bot logged work on LOG4J2-2899:
------------------------------------------
Author: ASF GitHub Bot
Created on: 19/Jul/20 02:57
Start Date: 19/Jul/20 02:57
Worklog Time Spent: 10m
Work Description: rgoers edited a comment on pull request #390:
URL: https://github.com/apache/logging-log4j2/pull/390#issuecomment-660576958
Yeah, I don't think it can be guaranteed that the methods will always be
called on the same thread. But I am also noticing that the method stores the
thread it found but then doesn't returned it if it was cached.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 460775)
Time Spent: 40m (was: 0.5h)
> LogEventWrapper.getThread only works on the first invocation
> ------------------------------------------------------------
>
> Key: LOG4J2-2899
> URL: https://issues.apache.org/jira/browse/LOG4J2-2899
> Project: Log4j 2
> Issue Type: Bug
> Components: log4j 1.2 emulation
> Affects Versions: 2.13.3
> Reporter: Carter Kozak
> Assignee: Carter Kozak
> Priority: Major
> Time Spent: 40m
> Remaining Estimate: 0h
>
> LogEventWrapper.getThread attempts to memoize the thread value but returns
> null if the thread field is already set where it should return the field
> value.
>
> Additionally the thread is discovered using Thread.getAllStackTraces which is
> an expensive operation. In most cases it should be sufficient to check only
> the current thread, if the getThread method is called from another thread
> there's no guarantee that the original thread still uses the same name, or
> that it still exists.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
