When checking for datetime field overflow, we should allow a fractional-second
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 1 May 2009 19:29:07 +0000 (19:29 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 1 May 2009 19:29:07 +0000 (19:29 +0000)
commited21fa0f5297582c3e8410b5a0ebd8830453799e
tree78338d8c3e24cdca93cba06d6005f29f9a04e996
parent4d6c79bc8ab7ac85e9c674b96c042699f1e01c91
When checking for datetime field overflow, we should allow a fractional-second
part that rounds up to exactly 1.0 second.  The previous coding rejected input
like "00:12:57.9999999999999999999999999999", with the exact number of nines
needed to cause failure varying depending on float-timestamp option and
possibly on platform.  Obviously this should round up to the next integral
second, if we don't have enough precision to distinguish the value from that.
Per bug #4789 from Robert Kruus.

In passing, fix a missed check for fractional seconds in one copy of the
"is it greater than 24:00:00" code.

Broken all the way back, so patch all the way back.
src/backend/utils/adt/datetime.c