nak() should follow backoff

[tamis-laan]

Proposed change

Make msg.nak() follow the set backoff strategy in the consumer.

Use case

Given the following:

# Backoff strategy
backoff = [2,4,8,16,32]

# Consumer configuration
config = ConsumerConfig(
    durable_name="action_workers",
    backoff=backoff,
    ack_wait=backoff[-1],
    max_deliver=len(backoff) + 1
)

# Subscribe
sub = await js.pull_subscribe("action.process.*", durable="action_workers", config=config)

while True:
    msgs = await sub.fetch(1)
    for msg in msgs:
        print(msg)
        msg.nak()

I would expect msg.nak() to make use of the backoff strategy. However backoff is only used for retries.

Contribution

No response

[Jarema]

nak marks message for immediate redelivery, so it does not apply ack_wait, or backoff, which is used in case of user not sending back anything.

You can however specify nak(delay) to achieve what I think would solve your use case.