Re: Minor refactor of the code in ExecScanExtended()

From: Aditya Gollamudi <adigollamudi(at)gmail(dot)com>
To: cca5507 <cca5507(at)qq(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Minor refactor of the code in ExecScanExtended()
Date: 2026-01-19 04:50:41
Message-ID: CAD-KL_G5Z_NbEkJcZhZu+ccWn39R9Sg9Zi3FZoKN_fRdNWDGnA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Jan 18, 2026 at 7:31 PM cca5507 <cca5507(at)qq(dot)com> wrote:

> Hi,
>
> The current code:
>
> if (!qual && !projInfo)
> {
> ResetExprContext(econtext);
> return ExecScanFetch(node, epqstate, accessMtd, recheckMtd);
> }
>
> ResetExprContext(econtext);
>
> The following format might be simpler:
>
> ResetExprContext(econtext);
>
> if (!qual && !projInfo)
> return ExecScanFetch(node, epqstate, accessMtd, recheckMtd);
>
> Attach a patch to do this.
>
> --
> Regards,
> ChangAo Chen
>
> Hi,

+1, seems like a simple refactor.
Tests are passing locally for me!

- Adi Gollamudi

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message vignesh C 2026-01-19 04:52:44 Re: Proposal: Conflict log history table for Logical Replication
Previous Message shveta malik 2026-01-19 04:41:02 Re: Simplify code building the LR conflict messages