Skip to content

Commit d791d92

Browse files
committed
Properly indicate sorted column
1 parent c141129 commit d791d92

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

test/TransitivePathTest.cpp

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,9 @@ class TransitivePathTest
5656
auto [T, qec] = makePath(std::move(input), vars, std::move(left),
5757
std::move(right), minDist, maxDist);
5858
auto leftOp = ad_utility::makeExecutionTree<ValuesForTesting>(
59-
qec, std::move(sideTable), sideVars, false, std::vector<ColumnIndex>{},
60-
LocalVocab{}, std::nullopt, forceFullyMaterialized);
59+
qec, std::move(sideTable), sideVars, false,
60+
std::vector<ColumnIndex>{sideTableCol}, LocalVocab{}, std::nullopt,
61+
forceFullyMaterialized);
6162
return T->bindLeftSide(leftOp, sideTableCol);
6263
}
6364

@@ -68,7 +69,8 @@ class TransitivePathTest
6869
auto [T, qec] = makePath(std::move(input), vars, std::move(left),
6970
std::move(right), minDist, maxDist);
7071
auto leftOp = ad_utility::makeExecutionTree<ValuesForTesting>(
71-
qec, std::move(sideTables), sideVars);
72+
qec, std::move(sideTables), sideVars, false,
73+
std::vector<ColumnIndex>{sideTableCol});
7274
return T->bindLeftSide(leftOp, sideTableCol);
7375
}
7476

@@ -79,8 +81,9 @@ class TransitivePathTest
7981
auto [T, qec] = makePath(std::move(input), vars, std::move(left),
8082
std::move(right), minDist, maxDist);
8183
auto rightOp = ad_utility::makeExecutionTree<ValuesForTesting>(
82-
qec, std::move(sideTable), sideVars, false, std::vector<ColumnIndex>{},
83-
LocalVocab{}, std::nullopt, forceFullyMaterialized);
84+
qec, std::move(sideTable), sideVars, false,
85+
std::vector<ColumnIndex>{sideTableCol}, LocalVocab{}, std::nullopt,
86+
forceFullyMaterialized);
8487
return T->bindRightSide(rightOp, sideTableCol);
8588
}
8689

@@ -91,7 +94,8 @@ class TransitivePathTest
9194
auto [T, qec] = makePath(std::move(input), vars, std::move(left),
9295
std::move(right), minDist, maxDist);
9396
auto rightOp = ad_utility::makeExecutionTree<ValuesForTesting>(
94-
qec, std::move(sideTables), sideVars);
97+
qec, std::move(sideTables), sideVars, false,
98+
std::vector<ColumnIndex>{sideTableCol});
9599
return T->bindRightSide(rightOp, sideTableCol);
96100
}
97101

0 commit comments

Comments
 (0)