broken logic of simple_eval_resowner after CALL and COMMIT inside procedure - Mailing list pgsql-hackers

From Pavel Stehule
Subject broken logic of simple_eval_resowner after CALL and COMMIT inside procedure
Date
Msg-id CAFj8pRD-90xCen9cn298tnyrudfz-EWsvthk8zJ9zRjGzvAk2w@mail.gmail.com
Whole thread Raw
Responses Re: broken logic of simple_eval_resowner after CALL and COMMIT inside procedure  (Pavel Stehule <pavel.stehule@gmail.com>)
List pgsql-hackers
Hi

Inline handler creates simple_eval_resowner (without parent).

Inside plpgsql_estate_setup this value is assigned to estate->simple_eval_resowner

<-->if (simple_eval_resowner)
<--><-->estate->simple_eval_resowner = simple_eval_resowner;
<-->else
<--><-->estate->simple_eval_resowner = shared_simple_eval_resowner;

When we call procedure with inner COMMIT, then when "before_lxid != after_lxid" following code is
executed.

<--><-->estate->simple_eval_estate = NULL;
<--><-->estate->simple_eval_resowner = NULL;
<--><-->plpgsql_create_econtext(estate);

and

fragment from plpgsql_create_econtext

<-->/*
<--> * Likewise for the simple-expression resource owner.
<--> */
<-->if (estate->simple_eval_resowner == NULL)
<-->{
<--><-->if (shared_simple_eval_resowner == NULL)
<--><--><-->shared_simple_eval_resowner =
<--><--><--><-->ResourceOwnerCreate(TopTransactionResourceOwner,
<--><--><--><--><--><--><--><--><-->"PL/pgSQL simple expressions");
<--><-->estate->simple_eval_resowner = shared_simple_eval_resowner;
<-->}

In this case simple_eval_resowner from inline handler is overwritten and only shared_simple_eval_resowner will be used.

So is it "estate->simple_eval_resowner = NULL;" error (without other conditions)?

Regards

Pavel


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Feature improvement: can we add queryId for pg_catalog.pg_stat_activity view?
Next
From: Pavel Stehule
Date:
Subject: Re: broken logic of simple_eval_resowner after CALL and COMMIT inside procedure