From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
Cc: | PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Allow to_timestamp(float8) to convert float infinity to timestam |
Date: | 2016-03-30 01:47:00 |
Message-ID: | 7106.1459302420@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Michael Paquier <michael(dot)paquier(at)gmail(dot)com> writes:
> On Wed, Mar 30, 2016 at 10:13 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> I'm inclined to just drop the out-of-range test cases. They're not that
>> useful IMO, and alternate expected-files are a real PITA for maintenance.
> Hm. Actually, they are quite useful to check error boundaries, so why
> not just simplifying the error message to "timestamp out of range" and
> remove the value from it?
Meh. I realize that there are a lot of places where we just say
"timestamp out of range" rather than trying to give a specific value,
but it's really contrary to our message style guidelines to not print
the complained-of value. I think we should leave the ereport calls as-is
and remove the test cases; to do otherwise is putting the regression tests
ahead of users. The upper-boundary test is quite dubiously useful anyway,
because it has to test a value that's very far away from where the
boundary actually is in most builds.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2016-03-30 01:49:49 | Re: pgsql: Add new replication mode synchronous_commit = 'remote_apply'. |
Previous Message | Michael Paquier | 2016-03-30 01:43:42 | Re: pgsql: Add new replication mode synchronous_commit = 'remote_apply'. |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2016-03-30 01:49:25 | Re: Relation extension scalability |
Previous Message | Dilip Kumar | 2016-03-30 01:43:16 | Re: Move PinBuffer and UnpinBuffer to atomics |