|From:||Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>|
|To:||Robert Haas <robertmhaas(at)gmail(dot)com>|
|Cc:||Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, Stephen Frost <sfrost(at)snowman(dot)net>, Mark Dilger <hornschnorter(at)gmail(dot)com>, Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>|
|Subject:||Re: Boolean partitions syntax|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Fri, Feb 2, 2018 at 4:40 PM, Peter Eisentraut
> <peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
>> There might be other options, but one way to solve this would be to
>> treat partition bounds as a general expression in the grammar and then
>> check in post-parse analysis that it's a constant.
> Yeah -- isn't the usual way of handling this to run the user's input
> through eval_const_expressions and see if the result is constant?
Not sure we want to go quite that far: at minimum it would imply invoking
arbitrary stuff during a utility statement, which we generally try to
avoid. Still, copy-from-query does that, so we can certainly make it
work if we wish.
Perhaps more useful to discuss: would that truly be the semantics we want,
or should we just evaluate the expression and have done? It's certainly
arguable that "IN (random())" ought to draw an error, not compute some
random value and use that. But if you are insistent on partition bounds
being immutable in any strong sense, you've already got problems, because
e.g. a timestamptz literal's interpretation isn't necessarily fixed.
It's only after we've reduced the original input to Datum form that we
can make any real promises about the value not moving. So I'm not seeing
where is the bright line between "IN ('today')" and "IN (random())".
regards, tom lane
|Next Message||David Kohn||2018-02-02 23:17:49||Re: Query running for very long time (server hanged) with parallel append|
|Previous Message||Claudio Freire||2018-02-02 22:52:02||Re: [HACKERS] Vacuum: allow usage of more than 1GB of work mem|