On Mon, Nov 26, 2018 at 11:21 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Hm. We intentionally allow SQL functions to be marked as less mutable
> than their internals, because sometimes that's useful for tricking
> the planner. However, IIRC we don't inline when we see that's the
> case. Maybe there needs to be a similar restriction for parallelism
> markings.
Not sure what you have in mind here exactly. I think that the only
correct thing to do is to mark the parent with the least safe setting
that could apply. When you inline, you might find out that things are
safer than they initially looked, but I think by then it's too late to
change your mind because ...
> Alternatively, could we postpone the parallelism checks till after
> function inlining? Do we even make any before that?
... I believe the parallel-safety checks are done very early, and if
you decide that it's not safe to proceed with parallelism, you can't
really change your mind later.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
