dfa1 edited a comment on pull request #770:
URL: https://github.com/apache/logging-log4j2/pull/770#issuecomment-1073049642
@rgoers @carterkozak @garydgregory
*Current state*
- this PR is implementing the SLF4J's contract in a reasonable way (my
humble opinion)
- test coverage is good
- it is possible to create detached markers directly
- it is possible to detach an existing `Marker` (my use case with
`StructuredArg`) to avoid any processing by log4j
- detached markers can be filtered by name (checked with @carterkozak
in `MarkerFilter`)
- `Marker.isInstanceOf(Marker)` is not working properly for detached
markers because of identity comparison (this could be a problem for people
using detached markers with log4j)
I will be more than happy to:
- finish this PR, adding anything you think is important
- add these changes in `log4j-slf4j18-impl` too
- provide feedback, testing and bug fixing after merging
- help you to maintain this code
I don't see the point of using my time to maintain my "private" workarounds,
I would be happy (and honored too) to use my time to collaborate with you on
log4j 2.x instead! :)
D.
--
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.
To unsubscribe, e-mail: notifications-unsubscr...@logging.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
