remove some verbose comments and fix some style

RyanL1997 · RyanL1997 · commit 0ca0dc6a0fe3 · 2025-08-22T18:31:42.000-07:00
Signed-off-by: Jialiang Liang &lt;jiallian@amazon.com&gt;
diff --git a/core/src/main/java/org/opensearch/sql/expression/parse/RegexExpression.java b/core/src/main/java/org/opensearch/sql/expression/parse/RegexExpression.java
@@ -38,7 +38,6 @@ public RegexExpression(Expression sourceField, Expression pattern, Expression id
 
   @Override
   ExprValue parseValue(ExprValue value) throws ExpressionEvaluationException {
-    // Use stringValue() which will throw exception for non-string types
     String rawString = value.stringValue();
 
     // Extract the specific named group
@@ -47,7 +46,6 @@ ExprValue parseValue(ExprValue value) throws ExpressionEvaluationException {
     if (extracted != null) {
       return new ExprStringValue(extracted);
     }
-
     log.debug("failed to extract pattern {} from input ***", regexPattern.pattern());
     return new ExprStringValue("");
   }

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,6 @@ public RegexExpression(Expression sourceField, Expression pattern, Expression id`
`38`	`38`
`39`	`39`	`@Override`
`40`	`40`	`ExprValue parseValue(ExprValue value) throws ExpressionEvaluationException {`
`41`		`- // Use stringValue() which will throw exception for non-string types`
`42`	`41`	`String rawString = value.stringValue();`
`43`	`42`
`44`	`43`	`// Extract the specific named group`
`@@ -47,7 +46,6 @@ ExprValue parseValue(ExprValue value) throws ExpressionEvaluationException {`
`47`	`46`	`if (extracted != null) {`
`48`	`47`	`return new ExprStringValue(extracted);`
`49`	`48`	`}`
`50`		`-`
`51`	`49`	`log.debug("failed to extract pattern {} from input ***", regexPattern.pattern());`
`52`	`50`	`return new ExprStringValue("");`
`53`	`51`	`}`