[GitHub] [activemq-nms-amqp] cjwmorgan-sol commented on issue #9: [PoC] Testing toolkit
cjwmorgan-sol commented on issue #9: [PoC] Testing toolkit URL: https://github.com/apache/activemq-nms-amqp/pull/9#issuecomment-521632602 Hey I found that the provider sends transfer frames on a sender link as settled even when the link sender settled mode is Unsettled. This is against amqp spec. So far I've been able to find at least one broker that detachs the sender link when a non-persistent(settled) message is sent. Sender links that support fire and forget settled message shoul dhave a send settle mode of Mixed not Unsettled. 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [activemq-nms-amqp] cjwmorgan-sol commented on issue #9: [PoC] Testing toolkit
cjwmorgan-sol commented on issue #9: [PoC] Testing toolkit URL: https://github.com/apache/activemq-nms-amqp/pull/9#issuecomment-518693552 > +1 from me more tests the better, just needs a jira, and squash commits Same +1 for me. 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [activemq-nms-amqp] cjwmorgan-sol commented on issue #9: [PoC] Testing toolkit
cjwmorgan-sol commented on issue #9: [PoC] Testing toolkit URL: https://github.com/apache/activemq-nms-amqp/pull/9#issuecomment-516028082 Improving integration tests is a great idea. For the changes to fix issue 1. I noticed that the closed callback for sessions does not remove the amqpSession from the amqpConnection, unlike amqpComsumer. Removing the explicit call to close would leave references in the amqpConnection list for its amqpSessions. For issue 2. NMSConnectionExceptions should be produced instead of IllegalStateExceptions on remote connection failures. 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services