On 2026-Feb-25, Vishal Prasanna wrote:
> > It looks OK for me. In this case an argument should be reduced from
> > ReorderBufferResetTXN(), right? It is harmless because the function is a static one.
>
> Yes, the `specinsert` is no longer needed in
> `ReorderBufferResetTXN()`. Updated the patch where `specinsert`
> cleanup is now handled in the `PG_CATCH()` block of
> `ReorderBufferProcessTXN()`, so it is always freed before the if/else
> branch.
Please don't do this. Changing the argument list of an exported
function is an ABI break. That's an OK change to do in branch master
(to keep the interface clean), but for released branches it is not
welcome, because it causes problems for users that have extensions that
call the function and were compiled with its older definition.
--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/
