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

    https://github.com/apache/storm/pull/2241#discussion_r158679635
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/daemon/supervisor/BasicContainer.java
 ---
    @@ -747,6 +748,16 @@ protected String javaCmd(String cmd) {
             commandList.add("-Dstorm.conf.file=" + topoConfFile);
             commandList.add("-Dstorm.options=" + stormOptions);
             commandList.add("-Djava.io.tmpdir=" + workerTmpDir);
    +        if (_topoConf.get("attach.debugger") != null) { // TODO: Roshan: 
remove this after stabilization
    --- End diff --
    
    Friendly reminder: this should be addressed before merging.


---

Reply via email to