You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@reef.apache.org by "Markus Weimer (JIRA)" <ji...@apache.org> on 2015/08/05 21:05:04 UTC
[jira] [Resolved] (REEF-557) Move EnvironmentUtils.addAll to
ConfigurationModule.set and delete EnvironmentUtils.addClasspath
[ https://issues.apache.org/jira/browse/REEF-557?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Markus Weimer resolved REEF-557.
--------------------------------
Resolution: Fixed
Resolved via [#336|https://github.com/apache/incubator-reef/pull/336]
> Move EnvironmentUtils.addAll to ConfigurationModule.set and delete EnvironmentUtils.addClasspath
> ------------------------------------------------------------------------------------------------
>
> Key: REEF-557
> URL: https://issues.apache.org/jira/browse/REEF-557
> Project: REEF
> Issue Type: Task
> Components: Tang
> Reporter: Mariia Mykhailova
> Assignee: Mariia Mykhailova
> Priority: Minor
> Fix For: 0.12
>
>
> As part of REEF-497 we need to deprecate {{addAll}} and {{addClasspath}} methods of {{EnvironmentUtils}}. We want to replace {{addAll}} with a {{ConfigurationModule.set}} method which would accept both {{Iterable<T>}} and {{T...}} as parameters.
> The logic of {{addClasspath}} is not something we want to provide as an official library method. So we want to delete it and replace its usages in our code with {{addAll}} and explicit {{getAllClasspathJars()}} call.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)