Do not attempt to use DeleteMultiple except on elastic tables

Fixes #469
MarkMpn · May 12, 2024 · 0a2b862 · 0a2b862
1 parent 197d8b1
commit 0a2b862
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/MarkMpn.Sql4Cds.Engine/ExecutionPlan/DeleteNode.cs b/MarkMpn.Sql4Cds.Engine/ExecutionPlan/DeleteNode.cs
@@ -263,7 +263,9 @@ protected override ExecuteMultipleResponse ExecuteMultiple(DataSource dataSource
             if (!req.Requests.All(r => r is DeleteRequest))
                 return base.ExecuteMultiple(dataSource, org, meta, req);
 
-            if (meta.DataProviderId == DataProviders.ElasticDataProvider || dataSource.MessageCache.IsMessageAvailable(meta.LogicalName, "DeleteMultiple"))
+            if (meta.DataProviderId == DataProviders.ElasticDataProvider
+                // DeleteMultiple is only supported on elastic tables, even if other tables do define the message
+                /* || dataSource.MessageCache.IsMessageAvailable(meta.LogicalName, "DeleteMultiple")*/)
             {
                 // Elastic tables can use DeleteMultiple for better performance than ExecuteMultiple
                 var entities = new EntityReferenceCollection();