You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@activemq.apache.org by "cshannon (via GitHub)" <gi...@apache.org> on 2023/06/06 23:11:08 UTC

[GitHub] [activemq] cshannon opened a new pull request, #1014: AMQ-9262 - Fix network subscriptions for composite consumers

cshannon opened a new pull request, #1014:
URL: https://github.com/apache/activemq/pull/1014

   This fixes network subscriptions that are generated on demand when a consumer uses composite destinations. Before this fix conduit subscriptions didn't work correctly. This fix now splits up the composite dest and generates correct demand for each of the individual destinations.


-- 
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-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [activemq] cshannon merged pull request #1014: AMQ-9262 - Fix network subscriptions for composite consumers

Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon merged PR #1014:
URL: https://github.com/apache/activemq/pull/1014


-- 
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-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [activemq] cshannon commented on pull request #1014: AMQ-9262 - Fix network subscriptions for composite consumers

Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon commented on PR #1014:
URL: https://github.com/apache/activemq/pull/1014#issuecomment-1579584304

   @mattrpav - feel free to take a look (or not), it's not a very common issue many people will run into it. I have tested things pretty extensively already before pushing this patch.


-- 
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-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [activemq] cshannon commented on pull request #1014: AMQ-9262 - Fix network subscriptions for composite consumers

Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon commented on PR #1014:
URL: https://github.com/apache/activemq/pull/1014#issuecomment-1579581284

   I will let CI run but plan to merge this in tomorrow if everything looks good.


-- 
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-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org