mattrpav commented on PR #1172: URL: https://github.com/apache/activemq/pull/1172#issuecomment-2317498051
@cshannon agree. Whether it is called “tech preview” or “experimental” is not a big deal to me. This first pass is about getting some modules so unit/itests and profiling work can begin. Having a module allows other contributors start looking at it as well. This is sufficiently complex to get going, there is no user “accidentally” turns this on in production. I will post test tool commands and results before merging any of this. As far as refactoring goes, I agree. I don’t think any wholesale search+replace of synchronized is a safe approach for stability. I was thinking about modernization of the code base, and there are plenty of extension points we could leverage to test new code paths that are more VT-friendly using new impls vs refactor-in-place. TransportConnector PersistenceAdapter QueueFactory (new idea to return different queue impls based on policy entry) .. etc -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information, visit: https://activemq.apache.org/contact
