Hi Álvaro,

On Thu, Sep 10, 2026 at 6:21 PM Álvaro Herrera <[email protected]> wrote:
>
> Hi,
>
> Not critically important but note that the PopActiveSnapshot and
> InvalidateCatalogSnapshot call and the comment about "giving up if there
> is still an active or registered snapshot" have a close relationship; I
> think it would make sense to keep them together.  I would add your new
> test-and-ereport just before the PopActiveSnapshot() call, to avoid
> breaking that.

Good catch. +1 for doing that.

-- 
Regards,
Xuneng Zhou
HighGo Software Co., Ltd.


Reply via email to