ppkarwasz closed issue #2238: Avoid a slow exception catch in
JdkMapAdapterStringMap constructor
URL: https://github.com/apache/logging-log4j2/issues/2238
--
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
jdgenerix commented on issue #2238:
URL:
https://github.com/apache/logging-log4j2/issues/2238#issuecomment-1930297586
@ppkarwasz Thank you.
I did a test and it looks fine to me.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
ppkarwasz commented on issue #2238:
URL:
https://github.com/apache/logging-log4j2/issues/2238#issuecomment-1928962960
@jdgenerix,
Can you check if the latest snapshot in the Maven repo
`https://repository.apache.org/snapshots` solves your problem?
--
This is an automated message
ppkarwasz commented on issue #2238:
URL:
https://github.com/apache/logging-log4j2/issues/2238#issuecomment-1908748402
@jdgenerix,
After looking better at the code the native storage of
`DefaultThreadContextMap` is JDK's `Map`, so the only optimization possible is
to create an
jdgenerix commented on issue #2238:
URL:
https://github.com/apache/logging-log4j2/issues/2238#issuecomment-1908102284
@ppkarwasz I'm sorry, i don't really see the optimzation you saw with
`ReadOnlyThreadContextMap`.
--
This is an automated message from the Apache Git Service.
To respond
ppkarwasz commented on issue #2238:
URL:
https://github.com/apache/logging-log4j2/issues/2238#issuecomment-1907818021
@jdgenerix,
Thank you for this report. Can you supply a PR to correct this?
--
This is an automated message from the Apache Git Service.
To respond to the message,
ppkarwasz opened a new issue, #2238:
URL: https://github.com/apache/logging-log4j2/issues/2238
### Discussed in https://github.com/apache/logging-log4j2/discussions/2235
Originally posted by **jdgenerix** January 24, 2024
The
