Re: ExecRTCheckPerms() and many prunable partitions

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Greg Stark <stark(at)mit(dot)edu>
Cc: Amit Langote <amitlangote09(at)gmail(dot)com>, Zhihong Yu <zyu(at)yugabyte(dot)com>, Julien Rouhaud <rjuju123(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: ExecRTCheckPerms() and many prunable partitions
Date: 2022-04-05 20:21:40
Message-ID: CAApHDvqjNwoQeLYN5imUSA9uy479Lz7BR8hcYS2siRzj04=XGg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, 6 Apr 2022 at 02:27, Greg Stark <stark(at)mit(dot)edu> wrote:
>
> This is failing regression tests. I don't understand how this patch
> could be affecting this test though. Perhaps it's a problem with the
> json patches that were committed recently -- but they don't seem to be
> causing other patches to fail.

I think this will just be related to the useprefix =
list_length(es->rtable) > 1; in show_plan_tlist(). There's likely not
much point in keeping the RTE for the view anymore. IIRC it was just
there to check permissions. Amit has now added another way of doing
those.

David

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Geoghegan 2022-04-05 20:29:38 Re: should vacuum's first heap pass be read-only?
Previous Message Robert Haas 2022-04-05 20:10:38 Re: should vacuum's first heap pass be read-only?