codelipenghui edited a comment on issue #8484: URL: https://github.com/apache/pulsar/issues/8484#issuecomment-726057120
> I was just surprised by deadLetterPolicy() auto-setting ackTImeoutMillis to 30s and breaking my time-consuming message processing. And also confused by changes in #3014 implying that ackTImeoutMillis was necessary (but I now see that NACKs were implemented after the DLQ feature so makes sense why it was the default at that time). Oh, I see. The #3014 is merged before the NACK support. I also have no idea how to avoid incompatibility. Maybe we can add more documents for the dead letter policy to describe the default 30s ack timeout will remove from the subsequent release. I think the 30s ack timeout that the dead letter topic introduced overwrite the ack timeout of the consumer. So it's better to add a flag to indicate the ack timeout is set by user, the dead letter policy should not overwrite it. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
