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 2022/02/22 22:17:38 UTC

[pinot] branch master updated: In BrokerRequestToQueryContextConverter, remove unused filterExpressionContext (#8238)

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 ff21071  In BrokerRequestToQueryContextConverter, remove unused filterExpressionContext (#8238)
ff21071 is described below

commit ff210718d8f3dd5112a20461c882e974796c82f8
Author: Xiaotian (Jackie) Jiang <17...@users.noreply.github.com>
AuthorDate: Tue Feb 22 14:17:00 2022 -0800

    In BrokerRequestToQueryContextConverter, remove unused filterExpressionContext (#8238)
---
 .../request/context/utils/BrokerRequestToQueryContextConverter.java   | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/pinot-core/src/main/java/org/apache/pinot/core/query/request/context/utils/BrokerRequestToQueryContextConverter.java b/pinot-core/src/main/java/org/apache/pinot/core/query/request/context/utils/BrokerRequestToQueryContextConverter.java
index 70df3db..4f835c2 100644
--- a/pinot-core/src/main/java/org/apache/pinot/core/query/request/context/utils/BrokerRequestToQueryContextConverter.java
+++ b/pinot-core/src/main/java/org/apache/pinot/core/query/request/context/utils/BrokerRequestToQueryContextConverter.java
@@ -102,11 +102,9 @@ public class BrokerRequestToQueryContextConverter {
 
     // WHERE
     FilterContext filter = null;
-    ExpressionContext filterExpressionContext = null;
     Expression filterExpression = pinotQuery.getFilterExpression();
     if (filterExpression != null) {
-      filter = RequestContextUtils.getFilter(pinotQuery.getFilterExpression());
-      filterExpressionContext = RequestContextUtils.getExpression(filterExpression);
+      filter = RequestContextUtils.getFilter(filterExpression);
     }
 
     // GROUP BY

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