fixed bug: use own implementation of list_difference to avoid side affects

42penguins · 42penguins · commit 5f75c8a55b10 · 2014-02-18T11:38:56.000+01:00
diff --git a/src/backend/executor/execProcnode.c b/src/backend/executor/execProcnode.c
@@ -425,11 +425,29 @@ void
 InvalidateStatisticsForTables(List* oldTableOids)
 {
 	List* relevantTableOids = NULL;
-	ListCell* l;
-	relevantTableOids = list_difference(piggyback->tableOids, oldTableOids);
-	foreach (l, relevantTableOids)
+	ListCell* l1;
+	ListCell* l2;
+	//relevantTableOids = list_difference(piggyback->tableOids, oldTableOids);
+	foreach(l1, piggyback->tableOids)
 	{
-		int currentOid = *((int*)lfirst(l));
+		int isNewOid = 1;
+		foreach(l2, oldTableOids)
+		{
+			if (*((int*)lfirst(l1)) == *((int*)lfirst(l2)))
+			{
+				isNewOid = 0;
+				break;
+			}
+		}
+		if (isNewOid == 1)
+		{
+			lappend(relevantTableOids, l1);
+		}
+	}
+
+	foreach (l1, relevantTableOids)
+	{
+		int currentOid = *((int*)lfirst(l1));
 		InvalidateStatisticsForTable(currentOid);
 	}
 }
