gianm opened a new pull request #10942: URL: https://github.com/apache/druid/pull/10942
The issue arises when matching against a long selector on the left-hand side to a string typed Index on the right-hand side, and when that Index also returns true from areKeysUnique. In this case, IndexedTableJoinMatcher would generate a ConditionMatcher that implements matchSingleRow by calling findUniqueLong on the Index. This is inappropriate because the Index is actually string typed. The fix is to check the type of the Index before deciding how to implement the ConditionMatcher. The patch adds "testMatchSingleRowToUniqueStringIndex" to IndexedTableJoinMatcherTest, which explores this case. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
