use child on no targets

Author: elianddb

sql/analyzer/apply_foreign_keys.go

@@ -189,11 +189,8 @@ func applyForeignKeysToNodes(ctx *sql.Context, a *Analyzer, n sql.Node, cache *f
 		if n.HasExplicitTargets() {
 			return n.WithExplicitTargets(foreignKeyHandlers), transform.NewTree, nil
 		} else {
-			newNode, err := n.WithChildren(foreignKeyHandlers...)
-			if err != nil {
-				return nil, transform.SameTree, err
-			}
-			return newNode, transform.NewTree, nil
+			// For simple DELETEs, update the targets array with foreign key handlers.
+			return n.WithTargets(foreignKeyHandlers), transform.NewTree, nil
 		}
 	default:
 		return n, transform.SameTree, nil

sql/plan/delete.go

@@ -70,9 +70,21 @@ func (p *DeleteFrom) WithExplicitTargets(targets []sql.Node) *DeleteFrom {
 	return &copy
 }
 
+// WithTargets returns a new DeleteFrom node instance with the specified |targets| set, preserving the
+// hasExplicitTargets flag. This is used for simple DELETEs where targets need to be updated (e.g., with
+// foreign key handlers) without changing whether they were explicitly specified.
+func (p *DeleteFrom) WithTargets(targets []sql.Node) *DeleteFrom {
+	copy := *p
+	copy.targets = targets
+	return &copy
+}
+
 // GetDeleteTargets returns the sql.Nodes representing the tables from which rows should be deleted.
 func (p *DeleteFrom) GetDeleteTargets() []sql.Node {
-	return p.targets
+	if len(p.targets) > 0 {
+		return p.targets
+	}
+	return []sql.Node{p.Child}
 }
 
 // Schema implements the sql.Node interface.