Fix timestamp range handling in regression tests of modules/commit_ts/
authorMichael Paquier <michael@paquier.xyz>
Mon, 13 Jul 2020 01:54:26 +0000 (10:54 +0900)
committerMichael Paquier <michael@paquier.xyz>
Mon, 13 Jul 2020 01:54:26 +0000 (10:54 +0900)
commit5bfe6a3c485d3259f59fa2d2e1d34dea1a3baeba
tree8a8b23b48b9e38572552088f22436d7c29380a1c
parentea3e15d1691ec4cadc67f160cc91c7f237a705ae
Fix timestamp range handling in regression tests of modules/commit_ts/

Switching the regression tests to use tstzrange() has proved to not be a
good idea for environments where the timestamp precision is low, as
internal range checks exclude the upper bound.  So, if the commit
timestamp of a transaction matched with now() from the next query,
the test would fail.  This changes to use two bound checks instead of
the range function, where the upper bound is inclusive.

Per buildfarm member jacana.

Discussion: https://postgr.es/m/20200712122507.GD21680@paquier.xyz
src/test/modules/commit_ts/expected/commit_timestamp.out
src/test/modules/commit_ts/expected/commit_timestamp_1.out
src/test/modules/commit_ts/sql/commit_timestamp.sql