Really appreciated the prompt reply Gary! Would you also mind taking care of cherry-picking this into master once merged into release-2.x, please?
On Fri, Jul 9, 2021 at 4:49 PM Gary Gregory <[email protected]> wrote: > 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> >>> . >>> >>
