On Thu, Jun 25, 2026 at 6:30 PM Dilip Kumar <[email protected]> wrote: > > On Thu, Jun 25, 2026 at 4:57 PM Amit Kapila <[email protected]> wrote: > > > > > 2. > > @@ -2482,6 +2681,8 @@ DropSubscription(DropSubscriptionStmt *stmt, > > bool isTopLevel) > > deleteDependencyRecordsFor(SubscriptionRelationId, subid, false); > > deleteSharedDependencyRecordsFor(SubscriptionRelationId, subid, 0); > > > > + drop_sub_conflict_log_table(subid, subname, subconflictlogrelid); > > > > It would be better to drop the table before cleaning up the dependency > > record. Right now, it is okay even in current order because dependency > > removal is trying to remove where subid is depender. > > So whats your suggestion change it now or not? I feel either way is fine. >
Let's change now itself. -- With Regards, Amit Kapila.
