quanzhian commented on issue #1753: URL: https://github.com/apache/incubator-seatunnel/issues/1753#issuecomment-1114500371
@legendtkl I tried your idea. I'll tell you why I didn't use it, 1. The content parsing fails, and the parsing method needs to be extended 2. The syntax is inconsistent. Originally, only one SQL syntax was used for implementation, but now it has become a mixture of multiple grammars, which is a little nondescript 3. Poor readability and maintainability The reason why I use SQL like method to set the Flink configuration 1. In terms of parsing, you only need to parse the contents of ordinary SQL files 2. The configuration content is implemented in SQL syntax, which is consistent 3. Good readability and maintainability The reason why I use SQL like method to set the Flink configuration 1. In terms of parsing, you only need to parse the contents of ordinary SQL files 2. The configuration content is implemented in SQL syntax, which is consistent 3. Good readability and maintainability -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
