Re: [PATCH] OAuth: fix performance bug with stuck multiplexer events
Jacob Champion <jacob.champion@enterprisedb.com>
From: Jacob Champion <jacob.champion@enterprisedb.com>
To: Thomas Munro <thomas.munro@gmail.com>, Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>,
Daniel Gustafsson <daniel@yesql.se>, Peter Eisentraut <peter@eisentraut.org>
Date: 2025-08-07T20:04:17Z
Lists: pgsql-hackers
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
oauth: Always link with -lm for floor()
- e76738e59790 18.0 landed
- 52ecd05aeef8 19 (unreleased) landed
-
oauth: Add unit tests for multiplexer handling
- 4e1e417330d4 19 (unreleased) landed
- 1443b6c0eaa2 19 (unreleased) landed
-
oauth: Ensure unused socket registrations are removed
- 16b0c48583a5 18.0 landed
- 3d9c03429a82 19 (unreleased) landed
-
oauth: Remove expired timers from the multiplexer
- e507e08acf63 18.0 landed
- 1749a12f0d20 19 (unreleased) landed
-
oauth: Remove stale events from the kqueue multiplexer
- ff181d1f876f 18.0 landed
- ff5b0824b3b0 19 (unreleased) landed
-
oauth: Track total call count during a client flow
- 41aac1483a6c 18.0 landed
- 3e311664e497 19 (unreleased) landed
Attachments
- since-v3.diff.txt (text/plain)
- v4-0001-oauth-Remove-stale-events-from-the-kqueue-multipl.patch (application/octet-stream) patch v4-0001
- v4-0002-oauth-Ensure-unused-socket-registrations-are-remo.patch (application/octet-stream) patch v4-0002
- v4-0003-oauth-Remove-expired-timers-from-the-multiplexer.patch (application/octet-stream) patch v4-0003
- v4-0004-oauth-Track-total-call-count-during-a-client-flow.patch (application/octet-stream) patch v4-0004
- v4-0005-oauth-Add-unit-tests-for-multiplexer-handling.patch (application/octet-stream) patch v4-0005
On Thu, Aug 7, 2025 at 11:11 AM Jacob Champion <jacob.champion@enterprisedb.com> wrote: > Thank you so much for the reviews! Here is v4, with the feedback from both of you. 0001-0004 are planned for backport; 0005 is slated for master only. Thanks again for the reviews! --Jacob