Skip to content

ARTEMIS-1555 MQTT - Unsubscribe topic in cluster environment leaves r…#1718

Merged
asfgit merged 2 commits intoapache:masterfrom
raulvaldoleiros:remoteQueueSubscriptionRemoveConsumer
Dec 15, 2017
Merged

ARTEMIS-1555 MQTT - Unsubscribe topic in cluster environment leaves r…#1718
asfgit merged 2 commits intoapache:masterfrom
raulvaldoleiros:remoteQueueSubscriptionRemoveConsumer

Conversation

@raulvaldoleiros
Copy link
Copy Markdown

…eference to the consumer

@raulvaldoleiros
Copy link
Copy Markdown
Author

The commit message is too long to be included in the commit

@jbertram
Copy link
Copy Markdown
Contributor

The commit message is too long to be included in the commit

Please follow the commit recommendations in the Hacking Guide.

Also, please squash your commits and then push -f to your branch where the PR is based (i.e. Skiler:remoteQueueSubscriptionRemoveConsumer). More details here. GitHub will automatically pick up the changes on the PR branch.

@mtaylor
Copy link
Copy Markdown
Contributor

mtaylor commented Dec 15, 2017

@Skiler Good catch and nice work on the tests. @jbertram @Skiler I am going to merge. I will take care of the commit message and squash. Thanks.

@asfgit asfgit merged commit 328ece0 into apache:master Dec 15, 2017
asfgit pushed a commit that referenced this pull request Dec 15, 2017
@jbertram
Copy link
Copy Markdown
Contributor

@Skiler, nice work!

@raulvaldoleiros
Copy link
Copy Markdown
Author

@jbertram @mtaylor thanks =D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants