Skip to content

Commit 0a18065

Browse files
committed
spotless fix
Signed-off-by: Jialiang Liang <jiallian@amazon.com>
1 parent fde2273 commit 0a18065

1 file changed

Lines changed: 9 additions & 7 deletions

File tree

legacy/src/main/java/org/opensearch/sql/legacy/parser/SqlParser.java

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -406,12 +406,14 @@ public JoinSelect parseJoinSelect(SQLQueryExpr sqlExpr) throws SqlParseException
406406
}
407407

408408
/**
409-
* Validates that JOIN queries do not use aggregations (GROUP BY or aggregate functions).
410-
* This limitation is documented in the official OpenSearch SQL documentation.
409+
* Validates that JOIN queries do not use aggregations (GROUP BY or aggregate functions). This
410+
* limitation is documented in the official OpenSearch SQL documentation.
411411
*/
412-
private void validateJoinWithoutAggregations(MySqlSelectQueryBlock query) throws SqlParseException {
413-
String errorMessage = "JOIN queries do not support aggregations on the joined result. "
414-
+ "For more information, see https://docs.opensearch.org/latest/search-plugins/sql/limitation/#join-does-not-support-aggregations-on-the-joined-result";
412+
private void validateJoinWithoutAggregations(MySqlSelectQueryBlock query)
413+
throws SqlParseException {
414+
String errorMessage =
415+
"JOIN queries do not support aggregations on the joined result. For more information, see"
416+
+ " https://docs.opensearch.org/latest/search-plugins/sql/limitation/#join-does-not-support-aggregations-on-the-joined-result";
415417

416418
if (query.getGroupBy() != null && !query.getGroupBy().getItems().isEmpty()) {
417419
throw new SqlParseException(errorMessage);
@@ -427,8 +429,8 @@ private void validateJoinWithoutAggregations(MySqlSelectQueryBlock query) throws
427429
}
428430

429431
/**
430-
* Recursively checks if an SQL expression contains aggregate functions.
431-
* Uses the same AGGREGATE_FUNCTIONS set as the Select class for consistency.
432+
* Recursively checks if an SQL expression contains aggregate functions. Uses the same
433+
* AGGREGATE_FUNCTIONS set as the Select class for consistency.
432434
*/
433435
private boolean containsAggregateFunction(SQLExpr expr) {
434436
if (expr == null) {

0 commit comments

Comments
 (0)