[GitHub] spark pull request #19392: [SPARK-22169][SQL] table name with numbers and ch...

2017-10-02 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/19392#discussion_r142277890 --- Diff: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 --- @@ -510,11 +510,15 @@ rowFormat ;

[GitHub] spark pull request #19392: [SPARK-22169][SQL] table name with numbers and ch...

2017-10-02 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/19392#discussion_r142184575 --- Diff: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 --- @@ -510,11 +510,15 @@ rowFormat ;

[GitHub] spark pull request #19392: [SPARK-22169][SQL] table name with numbers and ch...

2017-10-01 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/19392#discussion_r142054949 --- Diff: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 --- @@ -510,11 +510,15 @@ rowFormat ;

[GitHub] spark pull request #19392: [SPARK-22169][SQL] table name with numbers and ch...

2017-09-29 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/19392#discussion_r141998045 --- Diff: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 --- @@ -510,11 +510,15 @@ rowFormat ;

[GitHub] spark pull request #19392: [SPARK-22169][SQL] table name with numbers and ch...

2017-09-29 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/19392#discussion_r141950046 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/PlanParserSuite.scala --- @@ -110,6 +110,7 @@ class PlanParserSuite

[GitHub] spark pull request #19392: [SPARK-22169][SQL] table name with numbers and ch...

2017-09-29 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/19392#discussion_r141949952 --- Diff: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 --- @@ -510,11 +510,15 @@ rowFormat ;

[GitHub] spark pull request #19392: [SPARK-22169][SQL] table name with numbers and ch...

2017-09-29 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/19392#discussion_r141949849 --- Diff: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 --- @@ -510,11 +510,15 @@ rowFormat ;

[GitHub] spark pull request #19392: [SPARK-22169][SQL] table name with numbers and ch...

2017-09-29 Thread cloud-fan
GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/19392 [SPARK-22169][SQL] table name with numbers and characters should parsed successfuly ## What changes were proposed in this pull request? By definition the table name in Spark can be