Skip to content

Conversation

@c3llus
Copy link
Contributor

@c3llus c3llus commented Nov 18, 2024

Background

Currently, NSQD does not support a dedicated max-defer-timeout configuration for deferred publish operations. Instead, deferred publish times are implicitly governed by the max-req-timeout configuration. ref

Changes Done

  1. new max-defer-timeout command line options
  2. default 1h for max-defer-timeout
  3. refer to the new max-defer-timeout instead of the old max-req-timeout when doing deferred publish, for:
    a. HTTP
    b. TCP
  4. implemented backward compatibility when max-defer-timeout is not explicitly set, it falls back to max-req-timeout

TODO

@c3llus c3llus marked this pull request as ready for review November 18, 2024 06:33
@c3llus
Copy link
Contributor Author

c3llus commented Apr 17, 2025

hi @mreiferson, just following up on this PR; would really appreciate your review when you have time!

Thanks!

@mreiferson
Copy link
Member

The implementation is fine in general, but a few things:

  1. Let's not rename "timeout" to "delay"
  2. For users who are currently setting --max-req-timeout and expecting it to effect defers, there's probably some backward compatibility we want to add here. Maybe, if --max-defer-timeout is not set and --max-req-timeout is, we set the defer timeout to the req timeout?

@c3llus
Copy link
Contributor Author

c3llus commented Jun 16, 2025

Thanks for the feedback, @mreiferson! :)

I’ve updated the naming to stick with timeout instead of delay. Also added backward compatibility, when --max-defer-timeout isn’t explicitly set, it falls back to --max-req-timeout like before.

Let me know if there’s anything else you’d like to see adjusted!

@c3llus c3llus changed the title nsqd: add support for configurable max-defer-delay for DPUB nsqd: add support for configurable max-defer-timeout for DPUB Jun 19, 2025
@mreiferson
Copy link
Member

This is great, thank you! Can you squash down the commits, please?

@c3llus c3llus force-pushed the feat/issue-1499/new-max-defer-delay branch from 187664b to d182ee2 Compare June 20, 2025 04:30
@c3llus
Copy link
Contributor Author

c3llus commented Jun 20, 2025

pushed the squashed commits. let me know if you'd like any further tweaks, @mreiferson! :)

@mreiferson mreiferson merged commit 9ea5071 into nsqio:master Jun 29, 2025
0 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants