You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/08/01 02:35:00 UTC
[jira] [Commented] (FLINK-7269) Refactor passing of dynamic
properties
[ https://issues.apache.org/jira/browse/FLINK-7269?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16108302#comment-16108302 ]
ASF GitHub Bot commented on FLINK-7269:
---------------------------------------
Github user zjureel commented on the issue:
https://github.com/apache/flink/pull/4415
@tillrohrmann Great, use `loadConfigurationWithDynamicProperties` instead of `loadConfiguration` and `setDynamicProperties` will be more concise, it sounds good to me, thanks
> Refactor passing of dynamic properties
> --------------------------------------
>
> Key: FLINK-7269
> URL: https://issues.apache.org/jira/browse/FLINK-7269
> Project: Flink
> Issue Type: Improvement
> Components: Configuration
> Affects Versions: 1.3.1
> Reporter: Till Rohrmann
> Assignee: Fang Yong
>
> In order to set dynamic properties when loading the {{Configuration}} via {{GlobalConfiguration.loadConfiguration}}, we currently set a static field in {{GlobalConfiguration}} which is read whenever we load the {{Configuration}}.
> I think this is not a good pattern I propose to remove this functionality. Instead we should explicitly add the dynamic properties to the loaded {{Configuration}} at start of the application.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)