| From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
|---|---|
| To: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
| Cc: | Pg Bugs <pgsql-bugs(at)postgresql(dot)org>, Andrew B <pgsql(dot)20(dot)drkshadow(at)spamgourmet(dot)com> |
| Subject: | Re: Snapshot leak warning with lo_export in subtransaction |
| Date: | 2021-10-18 14:40:35 |
| Message-ID: | 202110181440.fdiy6yrecsuk@alvherre.pgsql |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
On 2021-Oct-18, Heikki Linnakangas wrote:
> I tried that, but didn't like the result. It conflated the user-visible
> INV_READ/WRITE flags with the new internal-only flag.
True -- I also noticed this and had written in my earlier reply that we
could perhaps define this flag internally, not exposing it to the user.
But that seemed weird from the API definition perspective, so I removed
that phrase before sending.
> Thinking about this some more, I came up with the attached. It moves the
> responsibility of registering the snapshot from inv_api.c to the caller.
I like this patch much better.
> With that change, there's no need for a new option to inv_open(). The
> division of labor between be-fsstubs.c and inv_api.c has always been a bit
> blurry, I think that this makes it slightly more clear.
Agreed.
--
Álvaro Herrera Valdivia, Chile — https://www.EnterpriseDB.com/
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrew Dunstan | 2021-10-18 18:07:38 | Re: BUG #17212: pg_amcheck fails on checking temporary relations |
| Previous Message | Tom Lane | 2021-10-18 13:23:38 | Re: BUG #17229: Segmentation Fault after upgrading to version 13 |