You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ja...@apache.org on 2024/06/24 18:05:13 UTC

(pinot) branch master updated: Fix field spec data type for JSON predicate evaluator tests (#13448)

This is an automated email from the ASF dual-hosted git repository.

jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new b809ccd5cb Fix field spec data type for JSON predicate evaluator tests (#13448)
b809ccd5cb is described below

commit b809ccd5cbb6d471679fde3b1e2cdb3421d487f9
Author: Yash Mayya <ya...@gmail.com>
AuthorDate: Mon Jun 24 23:35:06 2024 +0530

    Fix field spec data type for JSON predicate evaluator tests (#13448)
---
 .../filter/predicate/NoDictionaryEqualsPredicateEvaluatorsTest.java   | 4 ++--
 .../filter/predicate/NoDictionaryInPredicateEvaluatorTest.java        | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/pinot-core/src/test/java/org/apache/pinot/core/operator/filter/predicate/NoDictionaryEqualsPredicateEvaluatorsTest.java b/pinot-core/src/test/java/org/apache/pinot/core/operator/filter/predicate/NoDictionaryEqualsPredicateEvaluatorsTest.java
index 1a96e3ae4f..ef5f63df73 100644
--- a/pinot-core/src/test/java/org/apache/pinot/core/operator/filter/predicate/NoDictionaryEqualsPredicateEvaluatorsTest.java
+++ b/pinot-core/src/test/java/org/apache/pinot/core/operator/filter/predicate/NoDictionaryEqualsPredicateEvaluatorsTest.java
@@ -257,11 +257,11 @@ public class NoDictionaryEqualsPredicateEvaluatorsTest {
 
     EqPredicate eqPredicate = new EqPredicate(COLUMN_EXPRESSION, jsonString);
     PredicateEvaluator eqPredicateEvaluator =
-        EqualsPredicateEvaluatorFactory.newRawValueBasedEvaluator(eqPredicate, FieldSpec.DataType.STRING);
+        EqualsPredicateEvaluatorFactory.newRawValueBasedEvaluator(eqPredicate, FieldSpec.DataType.JSON);
 
     NotEqPredicate notEqPredicate = new NotEqPredicate(COLUMN_EXPRESSION, jsonString);
     PredicateEvaluator neqPredicateEvaluator =
-        NotEqualsPredicateEvaluatorFactory.newRawValueBasedEvaluator(notEqPredicate, FieldSpec.DataType.STRING);
+        NotEqualsPredicateEvaluatorFactory.newRawValueBasedEvaluator(notEqPredicate, FieldSpec.DataType.JSON);
 
     Assert.assertTrue(eqPredicateEvaluator.applySV(jsonString));
     Assert.assertFalse(neqPredicateEvaluator.applySV(jsonString));
diff --git a/pinot-core/src/test/java/org/apache/pinot/core/operator/filter/predicate/NoDictionaryInPredicateEvaluatorTest.java b/pinot-core/src/test/java/org/apache/pinot/core/operator/filter/predicate/NoDictionaryInPredicateEvaluatorTest.java
index 64f32acc73..7f452ed2b6 100644
--- a/pinot-core/src/test/java/org/apache/pinot/core/operator/filter/predicate/NoDictionaryInPredicateEvaluatorTest.java
+++ b/pinot-core/src/test/java/org/apache/pinot/core/operator/filter/predicate/NoDictionaryInPredicateEvaluatorTest.java
@@ -300,11 +300,11 @@ public class NoDictionaryInPredicateEvaluatorTest {
 
     InPredicate inPredicate = new InPredicate(COLUMN_EXPRESSION, jsonValues);
     PredicateEvaluator inPredicateEvaluator =
-        InPredicateEvaluatorFactory.newRawValueBasedEvaluator(inPredicate, FieldSpec.DataType.STRING);
+        InPredicateEvaluatorFactory.newRawValueBasedEvaluator(inPredicate, FieldSpec.DataType.JSON);
 
     NotInPredicate notInPredicate = new NotInPredicate(COLUMN_EXPRESSION, jsonValues);
     PredicateEvaluator notInPredicateEvaluator =
-        NotInPredicateEvaluatorFactory.newRawValueBasedEvaluator(notInPredicate, FieldSpec.DataType.STRING);
+        NotInPredicateEvaluatorFactory.newRawValueBasedEvaluator(notInPredicate, FieldSpec.DataType.JSON);
 
     for (String value : jsonValueSet) {
       Assert.assertTrue(inPredicateEvaluator.applySV(value));


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org