clean

gateway-service-impl/src/main/java/org/hypertrace/gateway/service/common/QueryRequestContext.java

@@ -9,7 +9,6 @@
 // Hold some of request fields and mappings needed when parsing the query service response
 public class QueryRequestContext extends RequestContext {
   private final Map<String, FunctionExpression> aliasToFunctionExpressionMap = new HashMap<>();
-  private final Map<FunctionExpression, String> functionExpressionToAliasMap = new HashMap<>();
   private final long startTimeMillis;
   private final long endTimeMillis;
   private final Map<String, TimeAggregation> aliasToTimeAggregation = new HashMap<>();
@@ -25,11 +24,6 @@ public QueryRequestContext(
 
   public void mapAliasToFunctionExpression(String alias, FunctionExpression functionExpression) {
     aliasToFunctionExpressionMap.put(alias, functionExpression);
-    functionExpressionToAliasMap.put(functionExpression, alias);
-  }
-
-  public String getAliasForFunctionExpression(FunctionExpression functionExpression) {
-    return functionExpressionToAliasMap.get(functionExpression);
   }
 
   public boolean containsFunctionExpression(String alias) {

gateway-service-impl/src/main/java/org/hypertrace/gateway/service/common/util/ExpressionReader.java

@@ -10,31 +10,14 @@
 import java.util.stream.Collectors;
 import org.hypertrace.gateway.service.v1.common.Expression;
 import org.hypertrace.gateway.service.v1.common.FunctionExpression;
-import org.hypertrace.gateway.service.v1.common.OrderByExpression;
 
 public class ExpressionReader {
-  public static List<OrderByExpression> getFunctionOrderByExpressions(
-      List<OrderByExpression> expressions) {
-    return expressions.stream()
-        .filter(
-            expression ->
-                expression.getExpression().getValueCase() == Expression.ValueCase.FUNCTION)
-        .collect(Collectors.toUnmodifiableList());
-  }
-
   public static List<Expression> getFunctionExpressions(List<Expression> expressions) {
     return expressions.stream()
         .filter(expression -> expression.getValueCase() == Expression.ValueCase.FUNCTION)
         .collect(Collectors.toUnmodifiableList());
   }
 
-  public static List<OrderByExpression> getAttributeOrderByExpressions(
-      List<OrderByExpression> expressions) {
-    return expressions.stream()
-        .filter(expression -> isAttributeSelection(expression.getExpression()))
-        .collect(Collectors.toUnmodifiableList());
-  }
-
   public static List<Expression> getAttributeExpressions(List<Expression> expressions) {
     return expressions.stream()
         .filter(ExpressionReader::isAttributeSelection)

gateway-service-impl/src/main/java/org/hypertrace/gateway/service/explore/ExploreService.java

@@ -26,12 +26,9 @@
 import org.hypertrace.gateway.service.explore.entity.EntityServiceEntityFetcher;
 import org.hypertrace.gateway.service.v1.explore.ExploreRequest;
 import org.hypertrace.gateway.service.v1.explore.ExploreResponse;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class ExploreService {
 
-  private static final Logger log = LoggerFactory.getLogger(ExploreService.class);
   private final GatewayServiceConfig gatewayServiceConfig;
   private final AttributeMetadataProvider attributeMetadataProvider;
   private final ExploreRequestValidator exploreRequestValidator = new ExploreRequestValidator();