Skip to content

Commit 9dfded5

Browse files
committed
Remove incorrect SetIndirect call - indirect already set via WithIndirect
The CTE indirect is already retrieved at line 663 and set on ExtendedTableMetadata via WithIndirect(indirect) at line 720. The SetIndirect call was incorrect as HeirarchyObjects.SetIndirect() expects RelationDTO, not astindirect.Indirect.
1 parent f467628 commit 9dfded5

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

internal/stackql/router/parameter_router.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -602,9 +602,6 @@ func (pr *standardParameterRouter) route(
602602
tableName := taxonomy.GetTableNameFromStatement(tb, pr.astFormatter)
603603
hIDs := internaldto.NewHeirarchyIdentifiers("", "", tableName, "")
604604
hr = tablemetadata.NewHeirarchyObjects(hIDs, isAwait)
605-
// Set the CTE indirect on the hierarchy objects so it can be accessed
606-
// during query rewriting and execution.
607-
hr.SetIndirect(cteIndirect)
608605
} else {
609606
hr, err = taxonomy.GetHeirarchyFromStatement(handlerCtx, tb, notOnParams, false, isAwait)
610607
if err != nil {

0 commit comments

Comments
 (0)