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] DLQ handling code doesn't copy the message key to the DLQ message when RECONSUMETIMES limit is hit #23181

Closed
3 tasks done
lhotari opened this issue Aug 15, 2024 · 0 comments · Fixed by #23182
Closed
3 tasks done

[Bug] DLQ handling code doesn't copy the message key to the DLQ message when RECONSUMETIMES limit is hit #23181

lhotari opened this issue Aug 15, 2024 · 0 comments · Fixed by #23182
Assignees
@lhotari
Labels
area/client type/bug The PR fixed a bug or issue reported a bug

Comments

@lhotari
Copy link
Member

lhotari commented Aug 15, 2024

Search before asking

  • I searched in the issues and found nothing similar.

Read release policy

  • I understand that unsupported versions don't get bug fixes. I will attempt to reproduce the issue on a supported version of Pulsar client and Pulsar broker.

Version

any released version

Minimal reproduce step

PR will contain a test to reproduce

What did you expect to see?

message key should be copied in all cases

What did you see instead?

message key wasn't copied with reconsumeLater and RECONSUMETIMES exceeds deadLetterPolicy.getMaxRedeliverCount()

Anything else?

No response

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@lhotari lhotari added type/bug The PR fixed a bug or issue reported a bug area/client labels Aug 15, 2024
@lhotari lhotari self-assigned this Aug 15, 2024
@lhotari lhotari mentioned this issue Aug 15, 2024
Merged
4 tasks
lhotari added a commit that referenced this issue Aug 16, 2024
@lhotari
[fix][client] Copy orderingKey to retry letter topic and DLQ messages…
c83428f 
… and fix bug in copying (#23182)

Fixes #23173
Fixes #23181

See #23173 and #23181

- copy ordering key to messages sent to retry letter topic and DLQ topic

(cherry picked from commit 67fc5b9)
lhotari added a commit that referenced this issue Aug 16, 2024
@lhotari
[fix][client] Copy orderingKey to retry letter topic and DLQ messages…
b1ecec4 
… and fix bug in copying (#23182)

Fixes #23173
Fixes #23181

### Motivation

See #23173 and #23181

### Modifications

- copy ordering key to messages sent to retry letter topic and DLQ topic

(cherry picked from commit 67fc5b9)
lhotari added a commit that referenced this issue Aug 16, 2024
@lhotari
[fix][client] Copy orderingKey to retry letter topic and DLQ messages…
ea9b7c2 
… and fix bug in copying (#23182)

Fixes #23173
Fixes #23181

See #23173 and #23181

- copy ordering key to messages sent to retry letter topic and DLQ topic

(cherry picked from commit 67fc5b9)
lhotari added a commit that referenced this issue Aug 16, 2024
@lhotari
[fix][client] Copy orderingKey to retry letter topic and DLQ messages…
a3f6577 
… and fix bug in copying (#23182)

Fixes #23173
Fixes #23181

### Motivation

See #23173 and #23181

### Modifications

- copy ordering key to messages sent to retry letter topic and DLQ topic

(cherry picked from commit 67fc5b9)
nikhil-ctds pushed a commit to datastax/pulsar that referenced this issue Aug 16, 2024
@lhotari @nikhil-ctds
[fix][client] Copy orderingKey to retry letter topic and DLQ messages…
73d7b6d 
… and fix bug in copying (apache#23182)

Fixes apache#23173
Fixes apache#23181

See apache#23173 and apache#23181

- copy ordering key to messages sent to retry letter topic and DLQ topic

(cherry picked from commit 67fc5b9)
(cherry picked from commit c83428f)
nikhil-ctds pushed a commit to datastax/pulsar that referenced this issue Aug 16, 2024
@lhotari @nikhil-ctds
[fix][client] Copy orderingKey to retry letter topic and DLQ messages…
b9e08b6 
… and fix bug in copying (apache#23182)

Fixes apache#23173
Fixes apache#23181

See apache#23173 and apache#23181

- copy ordering key to messages sent to retry letter topic and DLQ topic

(cherry picked from commit 67fc5b9)
(cherry picked from commit c83428f)
srinath-ctds pushed a commit to datastax/pulsar that referenced this issue Aug 20, 2024
@lhotari @srinath-ctds
[fix][client] Copy orderingKey to retry letter topic and DLQ messages…
42e9157 
… and fix bug in copying (apache#23182)

Fixes apache#23173
Fixes apache#23181

See apache#23173 and apache#23181

- copy ordering key to messages sent to retry letter topic and DLQ topic

(cherry picked from commit 67fc5b9)
(cherry picked from commit c83428f)
grssam pushed a commit to grssam/pulsar that referenced this issue Sep 4, 2024
@lhotari @grssam
[fix][client] Copy orderingKey to retry letter topic and DLQ messages…
98aba50 
… and fix bug in copying (apache#23182)

Fixes apache#23173
Fixes apache#23181


### Motivation

See apache#23173 and apache#23181

### Modifications

- copy ordering key to messages sent to retry letter topic and DLQ topic
hanmz pushed a commit to hanmz/pulsar that referenced this issue Feb 12, 2025
@lhotari @hanmz
[fix][client] Copy orderingKey to retry letter topic and DLQ messages…
85ace77 
… and fix bug in copying (apache#23182)

Fixes apache#23173
Fixes apache#23181


### Motivation

See apache#23173 and apache#23181

### Modifications

- copy ordering key to messages sent to retry letter topic and DLQ topic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lhotari lhotari

Labels
area/client type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Milestone
No milestone
1 participant
@lhotari