Re: Minor refactor of the code in ExecScanExtended() - Mailing list pgsql-hackers

From Aditya Gollamudi
Subject Re: Minor refactor of the code in ExecScanExtended()
Date
Msg-id CAD-KL_G5Z_NbEkJcZhZu+ccWn39R9Sg9Zi3FZoKN_fRdNWDGnA@mail.gmail.com
Whole thread Raw
In response to Minor refactor of the code in ExecScanExtended()  ("cca5507" <cca5507@qq.com>)
List pgsql-hackers
On Sun, Jan 18, 2026 at 7:31 PM cca5507 <cca5507@qq.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 

pgsql-hackers by date:

Previous
From: shveta malik
Date:
Subject: Re: Simplify code building the LR conflict messages
Next
From: vignesh C
Date:
Subject: Re: Proposal: Conflict log history table for Logical Replication