Removed inadvertent changes

Author: suddendust

document-store/src/main/java/org/hypertrace/core/documentstore/expression/impl/AliasedIdentifierExpression.java

@@ -1,28 +1,31 @@
 package org.hypertrace.core.documentstore.expression.impl;
 
 import com.google.common.base.Preconditions;
-import lombok.EqualsAndHashCode;
 import lombok.Value;
 import org.hypertrace.core.documentstore.parser.SelectTypeExpressionVisitor;
 
 /**
  * Expression for referencing an identifier/column name within a context having an alias.
  *
  * <p>Example: In this query: <code>
- * SELECT item, quantity, date FROM <implicit_collection> JOIN ( SELECT item, MAX(date) AS
- * latest_date FROM <implicit_collection> GROUP BY item ) AS latest ON item = latest.item ORDER BY
- * `item` ASC;
+ *     SELECT item, quantity, date
+ *     FROM <implicit_collection>
+ *     JOIN (
+ *         SELECT item, MAX(date) AS latest_date
+ *         FROM <implicit_collection>
+ *         GROUP BY item
+ *     ) AS latest
+ *     ON item = latest.item
+ *     ORDER BY `item` ASC;
  * </code> the rhs of the join condition "latest.item" can be expressed as: <code>
- * AliasedIdentifierExpression.builder().name("item").alias("alias1").build() </code>
+ *  AliasedIdentifierExpression.builder().name("item").alias("alias1").build() </code>
  */
-@EqualsAndHashCode(callSuper = true)
 @Value
 public class AliasedIdentifierExpression extends IdentifierExpression {
-
   String contextAlias;
 
   private AliasedIdentifierExpression(final String name, final String contextAlias) {
-    super(name, null);
+    super(name);
     this.contextAlias = contextAlias;
   }
 
@@ -41,7 +44,6 @@ public static AliasedIdentifierExpressionBuilder builder() {
   }
 
   public static class AliasedIdentifierExpressionBuilder {
-
     private String name;
     private String contextAlias;
 

document-store/src/main/java/org/hypertrace/core/documentstore/expression/impl/IdentifierExpression.java

@@ -44,7 +44,6 @@ public static IdentifierExpression of(final String name) {
     return new IdentifierExpression(name);
   }
 
-  // Package-private: used internally by factory methods
   static IdentifierExpression of(final String name, final DataType dataType) {
     Preconditions.checkArgument(name != null && !name.isBlank(), "name is null or blank");
     return new IdentifierExpression(name, dataType);

document-store/src/main/java/org/hypertrace/core/documentstore/expression/impl/JsonIdentifierExpression.java

@@ -67,7 +67,7 @@ public static JsonIdentifierExpression of(
 
   protected JsonIdentifierExpression(
       String name, String columnName, List<String> jsonPath, JsonFieldType fieldType) {
-    super(name, null);
+    super(name);
     this.columnName = columnName;
     this.jsonPath = jsonPath;
     this.fieldType = fieldType;