From notifications at github.com Sun May 3 21:50:36 2026 From: notifications at github.com (Darafei Praliaskouski) Date: Sun, 03 May 2026 14:50:36 -0700 Subject: [Tile-serving] [openstreetmap/mod_tile] Configure renderd request queue limits (PR #498) Message-ID: ## Summary This replaces the old hard-coded dirty queue limit approach from https://github.com/openstreetmap/mod_tile/pull/152 with configurable `renderd.conf` queue limits. - add `request_queue_limit` and `dirty_queue_limit` options to the `[renderd]` section - preserve the previous compiled-in defaults for metatile and non-metatile builds - size the queue de-duplication index from configured queue capacity - document queue behavior in the manpage and add an operator tuning/service manual ## Context The old PR tried to make the dirty queue much larger globally. The review thread pointed out that this is deployment-specific, can make stale background work accumulate, and does not solve queue fairness. This version keeps the default behavior unchanged and lets operators opt into larger queues with rollout guidance. ## Testing ```sh cmake --build build --target gen_tile_test render_list renderd_config_test renderd_config_test_helper -j2 ctest --test-dir build -R '^(gen_tile_test|renderd_config_test)$' --output-on-failure ``` Both targeted tests pass locally. You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/mod_tile/pull/498 -- Commit Summary -- * feat(renderd): configure request queue limits -- File Changes -- M docs/man/renderd.conf.5 (15) A docs/renderd-queue-tuning.md (68) M etc/renderd/renderd.conf (2) M etc/renderd/renderd.conf.examples (4) M etc/renderd/renderd.conf.in (2) M includes/render_config.h (9) M includes/renderd.h (2) M includes/request_queue.h (3) M src/renderd.c (2) M src/renderd_config.c (16) M src/request_queue.c (53) M tests/gen_tile_test.cpp (44) M tests/renderd_config_test.cpp (47) M tests/renderd_config_test_helper.cpp (12) -- Patch Links -- https://github.com/openstreetmap/mod_tile/pull/498.patchhttps://github.com/openstreetmap/mod_tile/pull/498.diff -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/mod_tile/pull/498 You are receiving this because you are subscribed to this thread. Message ID: <openstreetmap/mod_tile/pull/498 at github.com> -------------- next part -------------- An HTML attachment was scrubbed... URL: From notifications at github.com Sun May 3 21:50:51 2026 From: notifications at github.com (Darafei Praliaskouski) Date: Sun, 03 May 2026 14:50:51 -0700 Subject: [Tile-serving] [openstreetmap/mod_tile] Update render queue limits (#152) In-Reply-To: References: Message-ID: Closed #152. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/mod_tile/pull/152#event-25113058888 You are receiving this because you are subscribed to this thread. Message ID: -------------- next part -------------- An HTML attachment was scrubbed... URL: From notifications at github.com Mon May 4 00:10:52 2026 From: notifications at github.com (Darafei Praliaskouski) Date: Sun, 03 May 2026 17:10:52 -0700 Subject: [Tile-serving] [openstreetmap/mod_tile] Configure renderd request queue limits (PR #498) In-Reply-To: References: Message-ID: @Komzpa pushed 1 commit. 28acc43264ee9bb8fa6f6589c46b64817dd317ed feat(renderd): use priority queue for render requests -- View it on GitHub: https://github.com/openstreetmap/mod_tile/pull/498/changes/c3c4bea969ef1cd2fb52b3d513c318e0414fb0b8..28acc43264ee9bb8fa6f6589c46b64817dd317ed You are receiving this because you are subscribed to this thread. Message ID: -------------- next part -------------- An HTML attachment was scrubbed... URL: