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.
---