[ 
https://issues.apache.org/jira/browse/APEXCORE-495?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15389949#comment-15389949
 ] 

ASF GitHub Bot commented on APEXCORE-495:
-----------------------------------------

Github user davidyan74 commented on a diff in the pull request:

    https://github.com/apache/apex-core/pull/360#discussion_r71920532
  
    --- Diff: 
engine/src/main/java/com/datatorrent/stram/client/ConfigPackage.java ---
    @@ -37,10 +37,13 @@
     
     import org.apache.commons.io.FileUtils;
     import org.apache.commons.lang3.StringUtils;
    +import org.apache.hadoop.conf.Configuration;
     
     import net.lingala.zip4j.core.ZipFile;
     import net.lingala.zip4j.exception.ZipException;
     
    +import static 
com.datatorrent.stram.client.StramClientUtils.jsonFileToAppInfo;
    --- End diff --
    
    jsonFileToAppInfo is only used in one place. We should probably just use 
StramClientUtils.jsonFileToAppInfo, instead of import static here, to be 
consistent with the rest of StramClientUtils calls.


> Enhancing the configuration package to store apps
> -------------------------------------------------
>
>                 Key: APEXCORE-495
>                 URL: https://issues.apache.org/jira/browse/APEXCORE-495
>             Project: Apache Apex Core
>          Issue Type: Improvement
>            Reporter: Sandesh
>            Assignee: Sandesh
>
> Apex supports configuration package, separates application package from the 
> actual configuration. (http://docs.datatorrent.com/configuration_packages/)
> We want to enhance the configuration package by adding support to "add Apps" 
> (json format). 
> UseCase: Multiple users sharing the same app package, but have a different 
> view of the golden copy of the app package.
> Note: This feature is requested by an Apex user.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to