kenliao94 commented on PR #1364:
URL: https://github.com/apache/activemq/pull/1364#issuecomment-2536928048

   Hi @cshannon so the goal here is to implement the behavior as close to the 
spec as possible. There are 9 items in the spec, so are we saying that only 
7.3.3 message orders is not met? 
   
   In another words:
   7.3.1 - met
   7.3.2 - met
   7.3.3 - not met
   7.3.4 - met 
   7.3.5 - no going to support it, met
   7.3.6 - not going to throw exception, it is application responsibility, met
   7.3.7 - met
   7.3.8 - I thought this is already met, but it seems like it has 
   7.3.9 - not going to throw exception, it is application responsibility, met
   
   
   The first iteration of this PR is to just wrap AsyncCallback into 
CompletionListener and have the CompletionListener implemented the same way as 
the original AsyncCallback. But as @tabish121 pointed out that version didn't 
implement the spec from the mailing list discussion 
https://lists.apache.org/thread/y2pz4y1p9c5l1cdm6lyd5spc810mk6kq we want to 
keep both behavior in ActiveMQ 6. Hence I created another PR that referenced 
the current design but added additional logic (wrapping user completion 
listener to perform check)
   
   I will do more testing and add more logic to try and meet 7.3.3. But would 
like to get clarity on whether this is the only one not meeting the spec or 
there are more. Thanks.


-- 
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


Reply via email to