cshannon commented on PR #1364: URL: https://github.com/apache/activemq/pull/1364#issuecomment-2548563408
@jbonofre - That's a great idea. As seen from the back and forth, the spec here is a bit tricky so I think it would be good to start a design doc or something to specify what the actual plan is for the changes. It's a bit hard to get onto the same page just from PR discussions and it can lead to miscommunication or different ideas for design. The main thing with this is we can all agree before code is changed so you don't have to throw away code or waste time etc. Generally speaking the design here has gotten way too complex I think for a first version of this feature so i think we need to work on clarifying what the plan is. So we should probably: 1. Start a new design doc and work with the community to gain consensus for the first version of this feature. 2. Open up a new PR based on that as this PR has gotten super long at this point. 3. Work on documenting future plans (supporting multiple requests at once without blocking and handling callback ordering, 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: gitbox-unsubscr...@activemq.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org For additional commands, e-mail: gitbox-h...@activemq.apache.org For further information, visit: https://activemq.apache.org/contact
