Hello, At Wed, 26 Aug 2026 18:36:00 -0700, Bharath Rupireddy <[email protected]> wrote in > > In short, having just the slot release in the subxact path gives the > > same error behavior, is simple to reason about, and fixes the crash > > reported in this thread. > > > > Thoughts? > > I removed the temporary slot handling in the code and tests and > attached the simplified v14 patch. I also adjusted the docs to mention > that temporary slots are not automatically dropped when an error is > raised and caught in a subtransaction.
Sorry for chiming in this late, but looking at the patch, I wonder whether this should be handled by ResourceOwner rather than AtEO(Sub)Xact. In particular, the fact that the ownership is simply transferred to the parent subtransaction on commit makes ResourceOwner seem like a natural fit here. What do you think? Regards, -- Kyotaro Horiguchi NTT Open Source Software Center
