[GitHub] [incubator-seatunnel] quanzhian commented on issue #1753: [Feature][seatunnal-flink-sql] Support dynamic config for Flink SQL mode job

[GitBox <gi...@apache.org>]

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: commits-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org