michaelandrepearce commented on PR #86: URL: https://github.com/apache/activemq-nms-amqp/pull/86#issuecomment-1470016855
@Jefwillems thanks, just need to squash the commits and adjust the commit message to be correctly formed and include the JIRA id. @Havret any objections here? it looks good to me. On review i noted that we're also missing REJECTED ack type, see here in java variant: https://github.com/apache/qpid-jms/blob/main/qpid-jms-client/src/main/java/org/apache/qpid/jms/provider/amqp/AmqpConsumer.java#L317 Any reason it was left out? -- 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]
