diff options
| author | Alexander Korotkov <akorotkov@postgresql.org> | 2019-10-21 23:04:14 +0300 |
|---|---|---|
| committer | Alexander Korotkov <akorotkov@postgresql.org> | 2019-10-21 23:07:07 +0300 |
| commit | 52ad1e659967896ed153185328ffe806d69abcb6 (patch) | |
| tree | 03ab5a5db0c8bac802160dcf538a38a4025f92c9 /src/include/utils/timestamp.h | |
| parent | a6888fde7f0dbe865559b31ba2ce01ac1150debe (diff) | |
| download | postgresql-52ad1e659967896ed153185328ffe806d69abcb6.tar.gz | |
Refactor jsonpath's compareDatetime()
This commit refactors come ridiculous coding in compareDatetime(). Also, it
provides correct cross-datatype comparison even when one of values overflows
during cast. That eliminates dilemma on whether we should suppress overflow
errors during cast.
Reported-by: Tom Lane
Discussion: https://postgr.es/m/32308.1569455803%40sss.pgh.pa.us
Discussion: https://postgr.es/m/a5629d0c-8162-7559-16aa-0c8390d6ba5f%40postgrespro.ru
Author: Nikita Glukhov, Alexander Korotkov
Diffstat (limited to 'src/include/utils/timestamp.h')
| -rw-r--r-- | src/include/utils/timestamp.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/include/utils/timestamp.h b/src/include/utils/timestamp.h index e884d4405f..7652b41eae 100644 --- a/src/include/utils/timestamp.h +++ b/src/include/utils/timestamp.h @@ -97,8 +97,8 @@ extern int timestamp_cmp_internal(Timestamp dt1, Timestamp dt2); /* timestamp comparison works for timestamptz also */ #define timestamptz_cmp_internal(dt1,dt2) timestamp_cmp_internal(dt1, dt2) -extern TimestampTz timestamp2timestamptz_opt_error(Timestamp timestamp, - bool *have_error); +extern TimestampTz timestamp2timestamptz_opt_overflow(Timestamp timestamp, + int *overflow); extern int isoweek2j(int year, int week); extern void isoweek2date(int woy, int *year, int *mon, int *mday); |