Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-23 Thread via GitHub
boring-cyborg[bot] commented on PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#issuecomment-2128322457 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-23 Thread via GitHub
luoyuxia merged PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@f

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-23 Thread via GitHub
WencongLiu commented on PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#issuecomment-2126952619 It seems that the test is canceled. Could you help me trigger it again? @luoyuxia -- This is an automated message from the Apache Git Service. To respond to the message, p

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-23 Thread via GitHub
WencongLiu commented on PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#issuecomment-2126933902 @luoyuxia I've updated the pull request. PTAL. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-23 Thread via GitHub
WencongLiu commented on code in PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#discussion_r1611568586 ## flink-connector-hive/src/test/java/org/apache/flink/connectors/hive/HiveDialectSupportSQL11ReservedKeywordAsIdentifierTest.java: ## @@ -0,0 +1,223 @@ +/

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-20 Thread via GitHub
luoyuxia commented on PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#issuecomment-2120403078 > Thanks for the update, +1 for merging. > > It seems that the CI failure is unrelated, @luoyuxia could you please help confirming this? Yeah. I think the CI failu

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-20 Thread via GitHub
luoyuxia commented on code in PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#discussion_r1594867337 ## flink-connector-hive/src/main/java/org/apache/flink/table/planner/delegation/hive/parse/HiveASTParser.g: ## @@ -721,6 +722,12 @@ import org.apache.flink.

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-20 Thread via GitHub
luoyuxia commented on code in PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#discussion_r1594867337 ## flink-connector-hive/src/main/java/org/apache/flink/table/planner/delegation/hive/parse/HiveASTParser.g: ## @@ -721,6 +722,12 @@ import org.apache.flink.

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-19 Thread via GitHub
WencongLiu commented on code in PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#discussion_r1606184817 ## flink-connector-hive/src/main/java/org/apache/flink/table/planner/delegation/hive/HiveParserConstants.java: ## @@ -30,4 +30,6 @@ public class HiveParser

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-19 Thread via GitHub
WencongLiu commented on code in PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#discussion_r1606184637 ## flink-connector-hive/src/test/java/org/apache/flink/connectors/hive/HiveDialectSupportSQL11ReservedKeywordAsIdentifierTest.java: ## @@ -0,0 +1,230 @@ +/

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-19 Thread via GitHub
WencongLiu commented on code in PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#discussion_r1606184227 ## flink-connector-hive/src/test/java/org/apache/flink/connectors/hive/HiveDialectSupportSQL11ReservedKeywordAsIdentifierTest.java: ## @@ -0,0 +1,230 @@ +/

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-19 Thread via GitHub
WencongLiu commented on code in PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#discussion_r1606183796 ## flink-connector-hive/src/test/java/org/apache/flink/connectors/hive/HiveDialectSupportSQL11ReservedKeywordAsIdentifierTest.java: ## @@ -0,0 +1,230 @@ +/

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-15 Thread via GitHub
reswqa commented on code in PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#discussion_r1602581492 ## flink-connector-hive/src/main/java/org/apache/flink/table/planner/delegation/hive/HiveParserConstants.java: ## @@ -30,4 +30,6 @@ public class HiveParserCons

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-15 Thread via GitHub
reswqa commented on code in PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#discussion_r1602576000 ## flink-connector-hive/src/main/java/org/apache/flink/table/planner/delegation/hive/HiveParserConstants.java: ## @@ -30,4 +30,6 @@ public class HiveParserCons

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-05-15 Thread via GitHub
reswqa commented on code in PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#discussion_r1602571412 ## flink-connector-hive/src/test/java/org/apache/flink/connectors/hive/HiveDialectSupportSQL11ReservedKeywordAsIdentifierTest.java: ## @@ -0,0 +1,230 @@ +/* +

Re: [PR] [FLINK-35221][hive] Support SQL 2011 reserved keywords as identifiers in HiveParser [flink-connector-hive]

2024-04-28 Thread via GitHub
boring-cyborg[bot] commented on PR #18: URL: https://github.com/apache/flink-connector-hive/pull/18#issuecomment-2081434598 Thanks for opening this pull request! Please check out our contributing guidelines. (https://flink.apache.org/contributing/how-to-contribute.html) -- This is an