From 39eb4098a8d945b4e26c88525d979c6335097397 Mon Sep 17 00:00:00 2001 From: Christoph Pirkl Date: Mon, 10 Jul 2023 10:06:08 +0200 Subject: [PATCH] Improve javadoc formatting --- .../dialects/rewriting/SqlGenerationVisitor.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/main/java/com/exasol/adapter/dialects/rewriting/SqlGenerationVisitor.java b/src/main/java/com/exasol/adapter/dialects/rewriting/SqlGenerationVisitor.java index 043c016..222a152 100644 --- a/src/main/java/com/exasol/adapter/dialects/rewriting/SqlGenerationVisitor.java +++ b/src/main/java/com/exasol/adapter/dialects/rewriting/SqlGenerationVisitor.java @@ -25,11 +25,11 @@ * {@link SqlDialect#getSqlGenerator(SqlGenerationContext)}. *

* - * Note on operator associativity and parenthesis generation: Currently we use parenthesis almost always. Without - * parenthesis, two SqlNode graphs with different semantic lead to "select 1 = 1 - 1 + 1". Also "SELECT NOT NOT TRUE" - * needs to be written as "SELECT NOT (NOT TRUE)" to work at all, whereas SELECT NOT TRUE works fine without - * parentheses. Currently we make inflationary use of parenthesis to to enforce the right semantic, but hopefully there - * is a better way. + * Note on operator associativity and parenthesis generation: Currently we almost always use parenthesis. Without + * parenthesis, two {@link SqlNode} graphs with different semantic lead to {@code select 1 = 1 - 1 + 1}. Also + * {@code SELECT NOT NOT TRUE} needs to be written as {@code SELECT NOT (NOT TRUE)} to work at all, whereas + * {@code SELECT NOT TRUE} works fine without parentheses. Currently we make inflationary use of parenthesis to enforce + * the right semantic, but hopefully there is a better way. */ public class SqlGenerationVisitor implements SqlNodeVisitor, SqlGenerator { private final SqlDialect dialect;