On Fri, Sep 2, 2022 at 8:37 AM Daniel Gustafsson <dan...@yesql.se> wrote:
> This had bitrotted a fair bit, attached is a rebase along with (mostly) > documentation fixes. 0001 adds a generic GUC for ignoring event triggers > and > 0002 adds the login event with event trigger support, and hooks it up to > the > GUC such that broken triggers wont require single-user mode. Moving the CF > entry back to Needs Review. > > -- > Daniel Gustafsson https://vmware.com/ > > Hi, For EventTriggerOnLogin(): + LockSharedObject(DatabaseRelationId, MyDatabaseId, 0, AccessExclusiveLock); + + tuple = SearchSysCacheCopy1(DATABASEOID, ObjectIdGetDatum(MyDatabaseId)); + + if (!HeapTupleIsValid(tuple)) + elog(ERROR, "cache lookup failed for database %u", MyDatabaseId); Should the lock be taken after the check (HeapTupleIsValid) is done ? Cheers