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

    https://github.com/apache/apex-core/pull/360#discussion_r71920876
  
    --- Diff: engine/src/main/java/com/datatorrent/stram/cli/ApexCli.java ---
    @@ -3715,6 +3718,50 @@ DTConfiguration 
getLaunchAppPackageProperties(AppPackage ap, ConfigPackage cp, L
         return launchProperties;
       }
     
    +  private List<AppInfo> getAppsFromPackageAndConfig(AppPackage ap, 
ConfigPackage cp, boolean onlyConfigApps)
    +  {
    +    if (cp == null) {
    +      return ap.getApplications();
    +    }
    +
    +    File src = new File(cp.tempDirectory(), "app");
    +    File dest = new File(ap.tempDirectory(), "app");
    +
    +    if (!src.exists()) {
    +      return ap.getApplications();
    +    }
    +
    +    if (onlyConfigApps) {
    +      try {
    +        FileUtils.cleanDirectory(dest);
    +      } catch (IOException e) {
    +        e.printStackTrace();
    +      }
    +    } else {
    +      for (File app: src.listFiles()) {
    --- End diff --
    
    Please check the style of your IDE. There should be a space before this 
colon. Also check all other occurrences.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to