Re: [PR] [Fix] DLQ Messages will lose system properties when sent from reconsumeLater() [pulsar-client-go]

["geniusjoe (via GitHub)" <gi...@apache.org>]

geniusjoe commented on code in PR #1392:
URL: https://github.com/apache/pulsar-client-go/pull/1392#discussion_r2257011912


##########
pulsar/consumer_partition.go:
##########
@@ -1603,6 +1603,26 @@ func (pc *partitionConsumer) dispatcher() {
 			if !pc.isSeeking.Load() {
 				if pc.dlq.shouldSendToDlq(&nextMessage) {
 					// pass the message to the DLQ router
+					// we need to create a new ConsumerMessage and add dlq related metadata properties

Review Comment:
   I do not find typo word "orinal", maybe I should ignore this suggestion.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org