[calcite] branch master updated: [CALCITE-35] More test cases to guard against providing a broken fix for parenthesized join (Muhammad Gelbana)

[hongze]

This is an automated email from the ASF dual-hosted git repository.

hongze pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/master by this push:
     new ee14e01  [CALCITE-35] More test cases to guard against providing a 
broken fix for parenthesized join (Muhammad Gelbana)
ee14e01 is described below

commit ee14e012ecbd775c9f845682ddf02bf65204a0f7
Author: Muhammad Gelbana <m.gelb...@gmail.com>
AuthorDate: Wed May 22 21:59:00 2019 +0200

    [CALCITE-35] More test cases to guard against providing a broken fix for 
parenthesized join (Muhammad Gelbana)
    
    Close apache/calcite#1228
---
 .../java/org/apache/calcite/sql/parser/SqlParserTest.java    | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git 
a/core/src/test/java/org/apache/calcite/sql/parser/SqlParserTest.java 
b/core/src/test/java/org/apache/calcite/sql/parser/SqlParserTest.java
index f145379..d04bde1 100644
--- a/core/src/test/java/org/apache/calcite/sql/parser/SqlParserTest.java
+++ b/core/src/test/java/org/apache/calcite/sql/parser/SqlParserTest.java
@@ -8768,6 +8768,18 @@ public class SqlParserTest {
             + "FROM \"doublequotedtable\""));
   }
 
+  @Test public void testParenthesizedSubQueries() {
+    final String expected = "SELECT *\n"
+        + "FROM (SELECT *\n"
+        + "FROM `TAB`) AS `X`";
+
+    final String sql1 = "SELECT * FROM (((SELECT * FROM tab))) X";
+    sql(sql1).ok(expected);
+
+    final String sql2 = "SELECT * FROM ((((((((((((SELECT * FROM 
tab)))))))))))) X";
+    sql(sql2).ok(expected);
+  }
+
   protected void checkDialect(SqlDialect dialect, String sql,
       Matcher<String> matcher) throws SqlParseException {
     final SqlParser parser = getDialectSqlParser(sql, dialect);