@xumingming agreed


Would it be better or worse to construct `jvmopts` by joining `JAR_JVM_OPTS` and `"-Dstorm.jar=" + jarfile` on a space?

```
jvmopts=[' '.join(filter(None, [JAR_JVM_OPTS, "-Dstorm.jar=" + jarfile]))]
```