Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Removing low level consumer #37

Open
nick-zh opened this issue Sep 16, 2020 · 0 comments · May be fixed by #40
Open

Removing low level consumer #37

nick-zh opened this issue Sep 16, 2020 · 0 comments · May be fixed by #40
Milestone
v2.0.0

Comments

@nick-zh
Copy link
Contributor

nick-zh commented Sep 16, 2020

In light of arnaud-lb/php-rdkafka#388, it makes sense for us to also remove support for the low level consumer. As stated in the original issue, it is bad practice and leads to a lot of confusion.
This will also make it easier for people coming from other languages to find it more consistent with previous implementations.
@nick-zh nick-zh added this to the v2.0.0 milestone Sep 16, 2020
@nick-zh nick-zh mentioned this issue Sep 24, 2020
Closed
@nick-zh nick-zh linked a pull request Sep 24, 2020 that will close this issue
Chore/remove low level consumer #40
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
v2.0.0
Development

Successfully merging a pull request may close this issue.

Chore/remove low level consumer jobcloud/php-kafka-lib
1 participant
@nick-zh