Em ter., 17 de jun. de 2025 às 10:09, Aleksander Alekseev < aleksan...@timescale.com> escreveu:
> Hi, > > >> We would be in serious trouble if RelationReloadNailed() could be > >> called for a non-existing relation. Wouldn't Assert() be more > >> appropriate? > > > > IMO, I think no. > > > > In all paths, this check is done, why would this be the only exception? > > We use Asserts() for cases that shouldn't happen in practice for > performance reasons. If you are concerned about performance in this case, you can rest assured. > Since this code doesn't crash. No, you can't be sure. > I suspect this is > one of such cases. Unless you are aware of a specific scenario that > makes the code crash of course. > ScanPgRelation really can fail, better make sure. best regards, Ranier Vilela
