Github user berngp commented on the pull request:

    https://github.com/apache/spark/pull/116#issuecomment-37884923
  
    @aarondav just let me know what makes sense and I'll do it.  Also please 
consider the usage of an _extended_ `-X` option set, following the convention 
that the JVM uses. e.g. `java -X` will display the extended arguments the JVM 
supports.
    
    Another point to discuss is the precedence of all these arguments. 
Currently `spark-class` will source `conf/spark-env.sh` which will override 
values set through the optional arguments that this change suggests.
    
    Some changes that I have not submitted for review but might be worth 
mentioning:
    
    ```
    --java-home             : The path to the java home, else it will use 
JAVA_HOME.
    --jvm-debug             : The port that the JVM debugger will bind to. By 
default the debugger is disabled.
    ```
    
    The `--jvm-debug` might be useful if you are developing and running Spark 
Shell with a `"local"` master.


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