On 3/4/21 5:30 PM, John Naylor wrote: > > On Tue, Mar 2, 2021 at 7:32 PM Tomas Vondra > <tomas.von...@enterprisedb.com <mailto:tomas.von...@enterprisedb.com>> > wrote: >> Ummm, in brin_minmax_multi_distance_timetz()? I don't think timetz can >> be infinite, no? I think brin_minmax_multi_distance_date you meant? > > In timestamp.c there are plenty of checks for TIMESTAMP_NOT_FINITE > for TimestampTz types and I assumed that was applicable here. >
I don't think so. The NOT_FINITE macros are defined only for timestamps and dates, not for TimeTzADT. So I think the current code is correct. regards -- Tomas Vondra EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
