[pinot] branch master updated: Remove big-decimal getters for lower/upper bound (#8712)

[jackie]

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 3bbae6ef61 Remove big-decimal getters for lower/upper bound (#8712)
3bbae6ef61 is described below

commit 3bbae6ef6145ca29af7f8d0f115263437eef63bd
Author: Xiaotian (Jackie) Jiang <17555551+jackie-ji...@users.noreply.github.com>
AuthorDate: Tue May 17 10:45:22 2022 -0700

    Remove big-decimal getters for lower/upper bound (#8712)
---
 .../predicate/RangePredicateEvaluatorFactory.java  | 32 ++++++++++++++++------
 1 file changed, 24 insertions(+), 8 deletions(-)

diff --git 
a/pinot-core/src/main/java/org/apache/pinot/core/operator/filter/predicate/RangePredicateEvaluatorFactory.java
 
b/pinot-core/src/main/java/org/apache/pinot/core/operator/filter/predicate/RangePredicateEvaluatorFactory.java
index 0baf33deca..b1e432abee 100644
--- 
a/pinot-core/src/main/java/org/apache/pinot/core/operator/filter/predicate/RangePredicateEvaluatorFactory.java
+++ 
b/pinot-core/src/main/java/org/apache/pinot/core/operator/filter/predicate/RangePredicateEvaluatorFactory.java
@@ -285,10 +285,16 @@ public class RangePredicateEvaluatorFactory {
       _upperInclusive = upperInclusive;
     }
 
+    /**
+     * Returns the inclusive lower bound of the range.
+     */
     public int getLowerBound() {
       return _lowerInclusive ? _lowerBound : _lowerBound + 1;
     }
 
+    /**
+     * Returns the inclusive upper bound of the range.
+     */
     public int getUpperBound() {
       return _upperInclusive ? _upperBound : _upperBound - 1;
     }
@@ -330,10 +336,16 @@ public class RangePredicateEvaluatorFactory {
       _upperInclusive = upperInclusive;
     }
 
+    /**
+     * Returns the inclusive lower bound of the range.
+     */
     public long getLowerBound() {
       return _lowerInclusive ? _lowerBound : _lowerBound + 1;
     }
 
+    /**
+     * Returns the inclusive upper bound of the range.
+     */
     public long getUpperBound() {
       return _upperInclusive ? _upperBound : _upperBound - 1;
     }
@@ -375,10 +387,16 @@ public class RangePredicateEvaluatorFactory {
       _upperInclusive = upperInclusive;
     }
 
+    /**
+     * Returns the inclusive lower bound of the range.
+     */
     public float getLowerBound() {
       return _lowerInclusive ? _lowerBound : _lowerBound + Float.MIN_VALUE;
     }
 
+    /**
+     * Returns the inclusive upper bound of the range.
+     */
     public float getUpperBound() {
       return _upperInclusive ? _upperBound : _upperBound - Float.MIN_VALUE;
     }
@@ -420,10 +438,16 @@ public class RangePredicateEvaluatorFactory {
       _upperInclusive = upperInclusive;
     }
 
+    /**
+     * Returns the inclusive lower bound of the range.
+     */
     public double getLowerBound() {
       return _lowerInclusive ? _lowerBound : _lowerBound + Double.MIN_VALUE;
     }
 
+    /**
+     * Returns the inclusive upper bound of the range.
+     */
     public double getUpperBound() {
       return _upperInclusive ? _upperBound : _upperBound - Double.MIN_VALUE;
     }
@@ -465,14 +489,6 @@ public class RangePredicateEvaluatorFactory {
       _upperInclusive = upperInclusive;
     }
 
-    public BigDecimal geLowerBound() {
-      return _lowerBound;
-    }
-
-    public BigDecimal getUpperBound() {
-      return _upperBound;
-    }
-
     @Override
     public DataType getDataType() {
       return DataType.BIG_DECIMAL;


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