Fix CalcitePPLJoinIT (opensearch-project#3369)

* Fix PredicateAnalyzer for in and notIn

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Change text field to keyword since we don't support push down for that type

Signed-off-by: Heng Qian <qianheng@amazon.com>

---------

Signed-off-by: Heng Qian <qianheng@amazon.com>

Author: qianheng-aws

integ-test/src/test/resources/indexDefinitions/occupation_index_mapping.json

@@ -5,7 +5,7 @@
         "type": "keyword"
       },
       "occupation": {
-        "type": "text"
+        "type": "keyword"
       },
       "country": {
         "type": "text"

integ-test/src/test/resources/indexDefinitions/state_country_index_mapping.json

@@ -17,7 +17,7 @@
         }
       },
       "country": {
-        "type": "text"
+        "type": "keyword"
       },
       "year": {
         "type": "integer"

opensearch/src/main/java/org/opensearch/sql/opensearch/request/PredicateAnalyzer.java

@@ -41,6 +41,7 @@
 import com.google.common.base.Throwables;
 import com.google.common.collect.Range;
 import java.util.ArrayList;
+import java.util.Collection;
 import java.util.GregorianCalendar;
 import java.util.LinkedHashMap;
 import java.util.List;
@@ -897,15 +898,15 @@ public QueryExpression isTrue() {
 
     @Override
     public QueryExpression in(LiteralExpression literal) {
-      Iterable<?> iterable = (Iterable<?>) literal.value();
-      builder = termsQuery(getFieldReference(), iterable);
+      Collection<?> collection = (Collection<?>) literal.value();
+      builder = termsQuery(getFieldReference(), collection);
       return this;
     }
 
     @Override
     public QueryExpression notIn(LiteralExpression literal) {
-      Iterable<?> iterable = (Iterable<?>) literal.value();
-      builder = boolQuery().mustNot(termsQuery(getFieldReference(), iterable));
+      Collection<?> collection = (Collection<?>) literal.value();
+      builder = boolQuery().mustNot(termsQuery(getFieldReference(), collection));
       return this;
     }
   }