thelabdude commented on a change in pull request #2644: URL: https://github.com/apache/lucene-solr/pull/2644#discussion_r816037662
########## File path: solr/core/src/test/org/apache/solr/handler/TestSQLHandler.java ########## @@ -2388,6 +2388,7 @@ public void testMultiValuedFieldHandling() throws Exception { update.add("id", String.valueOf(maxDocs)); // all multi-valued fields are null update.commit(cluster.getSolrClient(), COLLECTIONORALIAS); + expectResults("SELECT stringxmv, stringsx, booleans FROM $ALIAS WHERE stringxmv IN ('a') AND stringxmv IN ('b')", 10); Review comment: how is this working? is calcite just matching all rows here? I thought the bug here was that calcite was erasing the two IN's and then matching none ;-) -- 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. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org