I commented :-) G
On Fri, Jul 9, 2021, 10:19 Volkan Yazıcı <[email protected]> wrote: > Gentlemen, you look to be the ones who got their hands most dirty with > jmx/Server. PR looks simple and legit to me. That said, I would appreciate > it if you can take a second look and merge it. > > On Fri, Jul 9, 2021 at 4:09 PM Markus Spann <[email protected]> > wrote: > >> …e MBean re-initialization due to InstanceAlreadyExistsException >> ------------------------------ >> You can view, comment on, or merge this pull request online at: >> >> https://github.com/apache/logging-log4j2/pull/547 >> Commit Summary >> >> - LOG4J2-3121: log4j2 config modified at run-time may trigger >> incomplete MBean re-initialization due to InstanceAlreadyExistsException >> >> File Changes >> >> - *M* >> log4j-core/src/main/java/org/apache/logging/log4j/core/jmx/Server.java >> >> <https://github.com/apache/logging-log4j2/pull/547/files#diff-6d580ac583f23407a865542b6fd054982307948838490e0602eefdeab2a6b5bc> >> (8) >> >> Patch Links: >> >> - https://github.com/apache/logging-log4j2/pull/547.patch >> - https://github.com/apache/logging-log4j2/pull/547.diff >> >> — >> You are receiving this because you are subscribed to this thread. >> Reply to this email directly, view it on GitHub >> <https://github.com/apache/logging-log4j2/pull/547>, or unsubscribe >> <https://github.com/notifications/unsubscribe-auth/AAARTSKAEJSNSA4NOQS3QBDTW37IFANCNFSM5AC242UA> >> . >> >
