-
Notifications
You must be signed in to change notification settings - Fork 20
Add proper errors to promisified admin api #159
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🎉 All Contributor License Agreements have been signed. Ready to merge. |
CHANGELOG.md
Outdated
@@ -7,6 +7,7 @@ v0.5.0 is a limited availability feature release. It is supported for all usage. | |||
|
|||
1. Add support for an Admin API to delete records.(#141). | |||
2. Fixes an issue with unresolved raced Promises leaking in the consumer (#151). | |||
3. Add KafkaJS-compatible errors to promisified Admin API (createTopics, deleteGroups, deleteTopicRecords) (#). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add PR number
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just that comment, thanks
cef9420
to
1997e22
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Compatibility related changes relating to partial-success or failure cases for createTopics, deleteGroups, and deleteTopicRecords.