Re: Remove unnecessary relabel stripping - Mailing list pgsql-hackers

From Richard Guo
Subject Re: Remove unnecessary relabel stripping
Date
Msg-id CAMbWs495JTAvOCG4RYfychCBPACZ8g4DwW1a3oaYFnZvfafr2Q@mail.gmail.com
Whole thread Raw
In response to Re: Remove unnecessary relabel stripping  (Tomas Vondra <tomas.vondra@2ndquadrant.com>)
Responses Re: Remove unnecessary relabel stripping  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers

On Thu, Apr 30, 2020 at 8:11 AM Tomas Vondra <tomas.vondra@2ndquadrant.com> wrote:
On Wed, Apr 29, 2020 at 03:51:40PM +0800, Richard Guo wrote:
>Hi hackers,
>
>Per discussion in [1], we don't need to strip relabel for the expr
>explicitly before calling pull_varnos() to retrieve all mentioned
>relids. pull_varnos() would recurse into T_RelabelType nodes.
>
>Add a patch to remove that and simplify the code a bit.
>
>[1]
>https://www.postgresql.org/message-id/flat/CAMbWs48HF9f%3Dg%2BjSmmYBnWub9%2BWyg5Xh-FoqAnvqAspue5ypAw%40mail.gmail.com#b6e77e4c1ae67e2c5d97dce830b58037
>
>Thanks
>Richard

Thanks, I'll get this pushed (or something similar to this patch) soon.

Thanks.
 

FWIW it'd be better to send the patch to the original thread instead of
starting a new one.

Ah yes, you're right. Sorry for not doing so.

Thanks
Richard 

pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: [HACKERS] Restricting maximum keep segments by repslots
Next
From: "Jonathan S. Katz"
Date:
Subject: Re: Poll: are people okay with function/operator table redesign?