diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2024-04-21 13:46:20 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2024-04-21 13:46:20 -0400 |
| commit | 0e56b2b9447b3441fa0408680ec52c1cd0b717cd (patch) | |
| tree | f179eb5e7bdec9b1bf2a052169e4423f16ead856 | |
| parent | e4c76f5eca92c195abd65c838b1efbd67da0bc16 (diff) | |
| download | postgresql-0e56b2b9447b3441fa0408680ec52c1cd0b717cd.tar.gz postgresql-0e56b2b9447b3441fa0408680ec52c1cd0b717cd.zip | |
Make postgres_fdw request remote time zone 'GMT' not 'UTC'.
This should have the same results for all practical purposes.
The advantage of selecting 'GMT' is that it's guaranteed to work
even when the remote system's timezone database is missing
entries, because pg_tzset() hard-wires handling of that,
at least in 9.2 and later.
(It seems like it would be a good idea to similarly hard-wire
correct handling of 'UTC', but that'll be a little more invasive
than I want to consider back-patching. Leave that for another
day when we're not in feature freeze.)
Per trouble report from Adnan Dautovic. Back-patch to all
supported branches.
Discussion: https://postgr.es/m/465248.1712211585@sss.pgh.pa.us
| -rw-r--r-- | contrib/postgres_fdw/connection.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/contrib/postgres_fdw/connection.c b/contrib/postgres_fdw/connection.c index a23064a2c80..d0eb515b369 100644 --- a/contrib/postgres_fdw/connection.c +++ b/contrib/postgres_fdw/connection.c @@ -427,10 +427,12 @@ configure_remote_session(PGconn *conn) * anyway. However it makes the regression test outputs more predictable. * * We don't risk setting remote zone equal to ours, since the remote - * server might use a different timezone database. Instead, use UTC - * (quoted, because very old servers are picky about case). + * server might use a different timezone database. Instead, use GMT + * (quoted, because very old servers are picky about case). That's + * guaranteed to work regardless of the remote's timezone database, + * because pg_tzset() hard-wires it (at least in PG 9.2 and later). */ - do_sql_command(conn, "SET timezone = 'UTC'"); + do_sql_command(conn, "SET timezone = 'GMT'"); /* * Set values needed to ensure unambiguous data output from remote. (This |