On Fri, Sep 17, 2021 at 8:28 AM Fujii Masao <masao.fu...@oss.nttdata.com> wrote:
>
> On 2021/09/17 11:40, Zhihong Yu wrote:
> > +           goto fail;          /* Trouble clearing prepared statements */
> >
> > The label fail can be removed. Under the above condition,  
> > entry->changing_xact_state is still true. You can directly return.
>
> Thanks for the review! Yes, you're right. Attached the updated version of the 
> patch.

+1 for the code refactoring (1 file changed, 75 insertions(+), 102
deletions(-)).

The v2 patch looks good to me as is.

Regards,
Bharath Rupireddy.


Reply via email to