Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Cleanup DLQ on create-consumer-group error #332

Merged
merged 2 commits into from Nov 30, 2017
Merged

Cleanup DLQ on create-consumer-group error #332

merged 2 commits into from Nov 30, 2017

Conversation

kirg
Copy link
Contributor

@kirg kirg commented Nov 21, 2017

Found one more case we missed .. that's causing us to leak DLQs.

@coveralls
Copy link

coveralls commented Nov 22, 2017 •
edited

Coverage Status

Changes Unknown when pulling 9d1e328 on cleanup-dlq-dest into ** on master**.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 67.304% when pulling 606d0bc on cleanup-dlq-dest into 371b784 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 67.304% when pulling 606d0bc on cleanup-dlq-dest into 371b784 on master.

@thuningxu thuningxu merged commit 8390176 into master Nov 30, 2017
@thuningxu thuningxu deleted the cleanup-dlq-dest branch November 30, 2017 06:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@thuningxu thuningxu thuningxu approved these changes

@datoug datoug Awaiting requested review from datoug

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@kirg @coveralls @thuningxu