Addressed comments in PR 5.

MitchellGale · MitchellGale · commit f93a82d3c7d9 · 2023-08-09T09:25:39.000-07:00
Signed-off-by: Mitchell Gale &lt;Mitchell.Gale@improving.com&gt;
diff --git a/core/src/main/java/org/opensearch/sql/analysis/SelectExpressionAnalyzer.java b/core/src/main/java/org/opensearch/sql/analysis/SelectExpressionAnalyzer.java
@@ -73,7 +73,7 @@ public List<NamedExpression> visitAlias(Alias node, AnalysisContext context) {
    * The Alias could be
    *
    * <ol>
-   *   <li>1. SELECT name, AVG(age) FROM s BY name -> Project(Alias("name", expr), Alias("AVG(age)",
+   *   <li>SELECT name, AVG(age) FROM s BY name -> Project(Alias("name", expr), Alias("AVG(age)",
    *       aggExpr)) Agg(Alias("AVG(age)", aggExpr))
    *   <li>SELECT length(name), AVG(age) FROM s BY length(name) Project(Alias("name", expr),
    *       Alias("AVG(age)", aggExpr)) Agg(Alias("AVG(age)", aggExpr))
diff --git a/core/src/main/java/org/opensearch/sql/ast/dsl/AstDSL.java b/core/src/main/java/org/opensearch/sql/ast/dsl/AstDSL.java
@@ -249,23 +249,27 @@ public static Function function(String funcName, UnresolvedExpression... funcArg
   }
 
   /**
-   * CASE<br>
-   * &emsp; WHEN search_condition THEN result_expr<br>
-   * &emsp; [WHEN search_condition THEN result_expr] ...<br>
-   * &emsp; [ELSE result_expr]<br>
+   * <pre>
+   * CASE
+   *    WHEN search_condition THEN result_expr<br>
+   *    [WHEN search_condition THEN result_expr] ...
+   *    [ELSE result_expr]
    * END
+   * </pre>
    */
   public UnresolvedExpression caseWhen(UnresolvedExpression elseClause,
                                        When... whenClauses) {
     return caseWhen(null, elseClause, whenClauses);
   }
 
   /**
+   * <pre>
    * CASE case_value_expr
    *     WHEN compare_expr THEN result_expr
    *     [WHEN compare_expr THEN result_expr] ...
    *     [ELSE result_expr]
    * END
+   * </pre>
    */
   public UnresolvedExpression caseWhen(UnresolvedExpression caseValueExpr,
                                        UnresolvedExpression elseClause,

Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ public List<NamedExpression> visitAlias(Alias node, AnalysisContext context) {`
`73`	`73`	`* The Alias could be`
`74`	`74`	`*`
`75`	`75`	`* <ol>`
`76`		`- * <li>1. SELECT name, AVG(age) FROM s BY name -> Project(Alias("name", expr), Alias("AVG(age)",`
	`76`	`+ * <li>SELECT name, AVG(age) FROM s BY name -> Project(Alias("name", expr), Alias("AVG(age)",`
`77`	`77`	`* aggExpr)) Agg(Alias("AVG(age)", aggExpr))`
`78`	`78`	`* <li>SELECT length(name), AVG(age) FROM s BY length(name) Project(Alias("name", expr),`
`79`	`79`	`* Alias("AVG(age)", aggExpr)) Agg(Alias("AVG(age)", aggExpr))`