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

Reply via email to