Skip to content

Commit 5031df8

Browse files
xiaoxmengfacebook-github-bot
authored andcommitted
fix: Use probe side column name for input column type and update SS check usage (#16205)
Summary: Fix a bug in `IndexLookupJoin.cpp` where the lookup input column was incorrectly using `indexKeyName` instead of `probeKeyName` Replaces all `VELOX_CHECK*` macros with their `SS_CHECK*` equivalents in `SequenceStorageIndexSource.cpp` to use the Sequence Storage-specific exception handling macros for better error categorization and consistency. Reviewed By: breeze1990, ximyu Differential Revision: D92024520
1 parent bef4942 commit 5031df8

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

velox/exec/IndexLookupJoin.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,7 @@ void IndexLookupJoin::initLookupInput() {
311311
indexKeyType->toString(),
312312
probeKeyType->toString());
313313
addLookupInputColumn(
314-
indexKeyName,
314+
probeKeyName,
315315
probeKeyType,
316316
probeKeyChannel,
317317
lookupInputNames,

0 commit comments

Comments
 (0)