Skip to content

skipping onPartitionsRevoked during consumer.close() call#886

Merged
srinagaraj merged 4 commits intolinkedin:masterfrom
srinagaraj:partitions
Mar 9, 2022
Merged

skipping onPartitionsRevoked during consumer.close() call#886
srinagaraj merged 4 commits intolinkedin:masterfrom
srinagaraj:partitions

Conversation

@srinagaraj
Copy link
Copy Markdown
Collaborator

Adding functionality to skip onPartitionsRevoked call during consumer.close(). Else it would lead to duplicate calls to onPartitionsRevoked().
More details about this in comments in the PR #881

@vmaheshw
Copy link
Copy Markdown
Collaborator

vmaheshw commented Mar 8, 2022

Please fix the title of the pull request.

Copy link
Copy Markdown
Collaborator

@vmaheshw vmaheshw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly looks good.

@srinagaraj srinagaraj changed the title Partitions skipping onPartitionsRevoked during consumer.close call Mar 8, 2022
@srinagaraj srinagaraj changed the title skipping onPartitionsRevoked during consumer.close call skipping onPartitionsRevoked during consumer.close() call Mar 8, 2022
@srinagaraj srinagaraj merged commit 37544cf into linkedin:master Mar 9, 2022
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.

3 participants