On 08/28/2016 04:59 PM, Tom Lane wrote:
Tomas Vondra <tomas.von...@2ndquadrant.com> writes:
I'm however pretty sure this will have absolutely no impact on profiles.

Yeah, I do not believe that either.  Also, I think that the intent of the
existing code is to defend against bad parameters even in non-assert
builds.  Which might argue for turning these into test-and-elog rather
than asserts.

I agree. There's a fair number of contrib modules, and when they compile just fine people are unlikely to test them with assert-enabled builds. So they would not notice any issues, but it'd get broken as we've removed the code that fixes the values for them.

Tomas Vondra                  http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to