On Thu, May 28, 2026 at 4:38 AM Amit Kapila <[email protected]> wrote: > > On Tue, May 26, 2026 at 2:38 AM shveta malik <[email protected]> wrote: > > > > 2) > > drop_subscription_dependencies(): > > > > + conflictrelname = get_rel_name(subconflictlogrelid); > > > > We can actually have a sanity check that we got the CLT using the relid. > > Assert(conflictrelname != NULL); > > > > elog will suit this place better as this can't be a direct coding > mistake. I see that at other places we used elog. See > if (result == NULL) > elog(ERROR, "cache lookup failed for index %u", indexId);
Yes it make sense to report elog, I will change this. -- Regards, Dilip Kumar Google
