pulsar icon indicating copy to clipboard operation
pulsar copied to clipboard
verified publisher icon apache

PoolMessage and DLQ can not work together

Open wangjialing218 opened this issue 4 years ago • 5 comments

Describe the bug PoolMessage and DLQ can not work together

To Reproduce Steps to reproduce the behavior:

  1. Enable poolMessages and DLQ in consumer side
  2. Call consumer.negativeAcknowledge(message) and then call message.release()
  3. The message did not resend to DLQ topic.

Expected behavior The message should be sent to DLQ topic.

Additional context The negative ack msgs will be sent to DLQ topic in a scheduler triggerRedelivery, when we enable poolMessages, message.release() will clear MessageImpl immediately, this will cause processPossibleToDLQ() fail to get message id and resend to DLQ.

wangjialing218 avatar Dec 13 '21 08:12 wangjialing218

I'm working on this

wangjialing218 avatar Dec 14 '21 01:12 wangjialing218

The issue had no activity for 30 days, mark with Stale label.

github-actions[bot] avatar Feb 26 '22 01:02 github-actions[bot]

The issue had no activity for 30 days, mark with Stale label.

github-actions[bot] avatar May 30 '22 02:05 github-actions[bot]

@hangc0276 @wangjialing218 I noticed that the solution proposed in the pull request for this bug has not been merged yet. Could you please provide an update on its status and let us know when we can expect this bug to be resolved?

kamilchociej avatar Jul 17 '24 13:07 kamilchociej

It looks like the PR has one commit/change request and some conflicts. Until those are resolved, we cannot merge the PR.

david-streamlio avatar Jul 25 '24 13:07 david-streamlio