On Wed, Aug 9, 2023 at 9:30 AM Junwang Zhao <zhjwpku(at)gmail(dot)com> wrote:
>
> In function `BackendXidGetPid`, when looping every proc's
> TransactionId, there is no need to access its PGPROC since there
> is shared memory access: `arrayP->pgprocnos[index]`.
>
> Though the compiler can optimize this kind of inefficiency, I
> believe we should ship with better code.
>
Looks good to me. However, I would just move the variable declaration
with their assignments inside the if () rather than combing the
expressions. It more readable that way.
--
Best Wishes,
Ashutosh Bapat