Skip to content

Commit 3545b1d

Browse files
authored
[native] Fix broken unit test due to IndexJoinNode change (prestodb#25691)
## Description Fix broken unit test due to prestodb#25668 ## Motivation and Context Note: Google Test filter = PrestoToVeloxQueryPlanTest.parseIndexJoinNode [==========] Running 1 test from 1 test suite. [----------] Global test environment set-up. [----------] 1 test from PrestoToVeloxQueryPlanTest [ RUN ] PrestoToVeloxQueryPlanTest.parseIndexJoinNode terminate called after throwing an instance of 'facebook::presto::protocol::OutOfRange' what(): [json.exception.out_of_range.403] key 'lookupVariables' not found IndexJoinNode List<VariableReferenceExpression> lookupVariables ## Impact No Impact. ## Release Notes Please follow [release notes guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines) and fill in the release notes below. ``` == NO RELEASE NOTE == ```
1 parent 91f1529 commit 3545b1d

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

presto-native-execution/presto_cpp/main/types/tests/PrestoToVeloxQueryPlanTest.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,8 @@ TEST_F(PrestoToVeloxQueryPlanTest, parseIndexJoinNode) {
235235
"form": "AND",
236236
"returnType": "boolean"
237237
},
238-
"type": "INNER"
238+
"type": "INNER",
239+
"lookupVariables": []
239240
}
240241
)";
241242

0 commit comments

Comments
 (0)