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

[Bug] Kafka Consumer Closing Twice #3307

Closed
2 tasks done
Markliniubility opened this issue Mar 1, 2023 · 0 comments · Fixed by #3308
Closed
2 tasks done

[Bug] Kafka Consumer Closing Twice #3307

Markliniubility opened this issue Mar 1, 2023 · 0 comments · Fixed by #3308
Assignees
@Markliniubility
Labels
bug Something isn't working
Milestone
1.9.0

Comments

@Markliniubility
Copy link
Member

Search before asking

  • I had searched in the issues and found no similar issues.

Environment

Mac

EventMesh version

master

What happened

It is possible that the Kafka consumer will be closed twice. It will raise an IllegalStateException when being closed again.

How to reproduce

  1. Run Eventmesh on Kafka
  2. Call shutdown to the consumer implementation

The exception will be raised.

Debug logs

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!
@Markliniubility Markliniubility added the bug Something isn't working label Mar 1, 2023
@Markliniubility Markliniubility mentioned this issue Mar 1, 2023
Merged
xwm1992 added a commit that referenced this issue Mar 2, 2023
@xwm1992
Merge pull request #3308 from apache/Kafka-Shutdown
984537d 
[ISSUE #3307] Kafka Consumer Shuts Down Properly
@xwm1992 xwm1992 added this to the 1.9.0 milestone Jun 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Markliniubility Markliniubility

Labels
bug Something isn't working
Projects
None yet
Milestone
1.9.0
Development

Successfully merging a pull request may close this issue.

[ISSUE #3307] Kafka Consumer Shuts Down Properly apache/eventmesh
2 participants
@xwm1992 @Markliniubility