You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@seatunnel.apache.org by GitBox <gi...@apache.org> on 2022/05/06 03:32:35 UTC

[GitHub] [incubator-seatunnel] ruanwenjun commented on pull request #1800: [Feature][Seatunnel-core-fline] Support Flink dynamic configurations

ruanwenjun commented on PR #1800:
URL: https://github.com/apache/incubator-seatunnel/pull/1800#issuecomment-1119230663

   > I think it is better to inject dynamic configurations when creating the Flink Environment than in the shell command. @ruanwenjun @legendtkl What do you think?
   
   I think we can use `-D` to implement this. You are right, inject configuration in creating environment is a better way, this way can check the config and do some other control, but this will need to do some complex change. As kelu said, we may need to pay attention to deal the different version.


-- 
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