supabase / wrappers / 29899015585

81%

Build Type

push

github

Committed by web-flow

Commit Message

fix(iceberg_fdw): [WRA-27] configurable HTTP timeouts for REST catalog (#617)

* fix(iceberg_fdw): add configurable HTTP timeouts for REST catalog

The Iceberg REST catalog client had no timeout, so a stalled connection
to the catalog server would hang the query indefinitely, uninterruptible
by statement_timeout or pg_cancel_backend, pinning VACUUM/replication
slot horizons in the process (WRA-27).

Adds request_timeout_ms/connect_timeout_ms server options and injects a
timeout-configured reqwest client into RestCatalogBuilder. This requires
bumping reqwest to 0.12 workspace-wide (with matching reqwest-middleware/
reqwest-retry/http bumps) since iceberg-catalog-rest pins reqwest 0.12.12.

* fix(iceberg_fdw): address Copilot review feedback

- Read request_timeout_ms/connect_timeout_ms from the vault-decrypted
  props map instead of raw server.options, so vault_-prefixed keys
  aren't silently ignored.
- Use an RFC 5737 TEST-NET address (192.0.2.1) instead of an RFC 1918
  address in the timeout test, since the latter can be routable in
  some CI/VPC environments.

Coverage Stats

16 of 17 new or added lines in 1 file covered. (94.12%)

1 existing line in 1 file now uncovered.

9486 of 11717 relevant lines covered (80.96%)

70.58 hits per line