Skip to content

Commit 67fe19e

Browse files
shim
1 parent e15f531 commit 67fe19e

1 file changed

Lines changed: 2 additions & 9 deletions

File tree

internal/stackql/dependencyplanner/dependencyplanner.go

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -113,17 +113,11 @@ func isAnnotationIndirection(annotationCtx taxonomy.AnnotationCtx) bool {
113113
// Unlike regular tables that need HTTP acquisition, indirections are already materialized by the
114114
// "create tail" builder and appear in the FROM clause as inline subqueries.
115115
// This method creates a NopBuilder as a placeholder in the execution DAG.
116+
// Indirections are NOT added to dp.tableSlice because they appear as inline subqueries
117+
// with their own internal control columns; adding them would generate incorrect outer WHERE clauses.
116118
func (dp *standardDependencyPlanner) orchestrateIndirection(
117119
annotationCtx taxonomy.AnnotationCtx,
118120
) (int, error) {
119-
rc, err := tableinsertioncontainer.NewTableInsertionContainer(
120-
annotationCtx.GetTableMeta(),
121-
dp.handlerCtx.GetSQLEngine(),
122-
dp.handlerCtx.GetTxnCounterMgr(),
123-
)
124-
if err != nil {
125-
return -1, err
126-
}
127121
builder := primitivebuilder.NewNopBuilder(
128122
dp.primitiveComposer.GetGraphHolder(),
129123
dp.primitiveComposer.GetTxnCtrlCtrs(),
@@ -133,7 +127,6 @@ func (dp *standardDependencyPlanner) orchestrateIndirection(
133127
)
134128
dp.execSlice = append(dp.execSlice, builder)
135129
idx := len(dp.execSlice) - 1
136-
dp.tableSlice = append(dp.tableSlice, rc)
137130
return idx, nil
138131
}
139132

0 commit comments

Comments
 (0)