On Fri, Jun 07, 2024 at 11:32:14AM +0800, jian he wrote:
> in deleteObjectsInList, under certain conditions trying to sort the to
> be deleted object list
> by just using sort_object_addresses seems to work,
> but it looks like a hack.
> maybe the proper fix would be in findDependentObjects.
@@ -1459,6 +1459,7 @@ RemoveRelations(DropStmt *drop)
[...]
- performMultipleDeletions(objects, drop->behavior, flags);
+ if (list_length(drop->objects) > 1)
+ sortable = false;
I have not studied the patch in details, but this looks
overcomplicated to me. All the callers of performMultipleDeletions
pass down sortable as true, while deleteObjectsInList() uses this
argument to avoid the sorting on nested calls. It seems to me that
this could be simpler.
--
Michael