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.

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

partitionProducer Send stuck #877

Closed
billowqiu opened this issue Oct 30, 2022 · 0 comments · Fixed by #1053
Closed

partitionProducer Send stuck #877

billowqiu opened this issue Oct 30, 2022 · 0 comments · Fixed by #1053

Comments

@billowqiu
Copy link
Contributor

billowqiu commented Oct 30, 2022

We use fork version in this line:https://github.com/billowqiu/pulsar-client-go/blob/1dc06ae37025b011ecc345312a65d797b436dedb/pulsar/producer_partition.go#L745

but the upstream version have the same problem.

Expected behavior

partitionProducer.Send should return after success or timeout.

Actual behavior

stuck in doneCh.

image

image

Steps to reproduce

How can we reproduce the issue

System configuration

Pulsar version: x.y

@billowqiu billowqiu changed the title Send callback stuck partitionProducer Send stuck Oct 30, 2022
RobertIndie pushed a commit that referenced this issue Jul 11, 2023
…ng as expected (#1053)

Fixes #877 

### Motivation

The original PR is #878. Because the original author @billowqiu has not continued to reply to the review comments for a long time, resubmit the fix here.

### Modifications

- Add select for ctx and doneCh in partitionProducer.Send()

---------

Co-authored-by: shenjiaqi.2769 <[email protected]>
RobertIndie pushed a commit that referenced this issue Sep 7, 2023
…ng as expected (#1053)

Fixes #877

### Motivation

The original PR is #878. Because the original author @billowqiu has not continued to reply to the review comments for a long time, resubmit the fix here.

### Modifications

- Add select for ctx and doneCh in partitionProducer.Send()

---------

Co-authored-by: shenjiaqi.2769 <[email protected]>
(cherry picked from commit be35740)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant