pgsql: Consistently declare timestamp variables as TimestampTz.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Consistently declare timestamp variables as TimestampTz.
Date: 2017-02-23 20:57:18
Message-ID: E1ch0Ry-0000zl-92@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Consistently declare timestamp variables as TimestampTz.

Twiddle the replication-related code so that its timestamp variables
are declared TimestampTz, rather than the uninformative "int64" that
was previously used for meant-to-be-always-integer timestamps.
This resolves the int64-vs-TimestampTz declaration inconsistencies
introduced by commit 7c030783a, though in the opposite direction to
what was originally suggested.

This required including datatype/timestamp.h in a couple more places
than before. I decided it would be a good idea to slim down that
header by not having it pull in <float.h> etc, as those headers are
no longer at all relevant to its purpose. Unsurprisingly, a small number
of .c files turn out to have been depending on those inclusions, so add
them back in the .c files as needed.

Discussion: https://postgr.es/m/26788.1487455319@sss.pgh.pa.us
Discussion: https://postgr.es/m/27694.1487456324@sss.pgh.pa.us

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/c29aff959dc64f7321062e7f33d8c6ec23db53d3

Modified Files
--------------
contrib/auth_delay/auth_delay.c | 2 ++
contrib/btree_gist/btree_ts.c | 2 ++
src/backend/access/common/reloptions.c | 2 ++
src/backend/access/transam/xlog.c | 1 +
src/backend/commands/prepare.c | 2 ++
src/backend/executor/nodeBitmapHeapscan.c | 2 ++
src/backend/replication/basebackup.c | 12 ++++-----
src/backend/replication/logical/worker.c | 14 +++++-----
src/backend/replication/walreceiver.c | 10 +++----
src/backend/replication/walsender.c | 9 ++++---
src/backend/tcop/pquery.c | 2 ++
src/backend/utils/adt/nabstime.c | 1 +
src/backend/utils/cache/inval.c | 2 ++
src/backend/utils/time/snapmgr.c | 45 ++++++++++++++++---------------
src/bin/pg_basebackup/pg_recvlogical.c | 18 ++++++-------
src/bin/pg_basebackup/receivelog.c | 22 +++++++--------
src/bin/pg_basebackup/streamutil.c | 20 +++++++-------
src/bin/pg_basebackup/streamutil.h | 7 ++---
src/include/datatype/timestamp.h | 4 ---
src/include/utils/snapmgr.h | 7 ++---
src/include/utils/snapshot.h | 3 ++-
src/include/utils/timestamp.h | 7 -----
22 files changed, 100 insertions(+), 94 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2017-02-24 03:08:44 pgsql: Add an Assert that enum_cmp_internal() gets passed an FmgrInfo p
Previous Message Tom Lane 2017-02-23 19:04:49 pgsql: Remove now-dead code for !HAVE_INT64_TIMESTAMP.