diff options
| author | Joshi <rekhajoshm@gmail.com> | 2015-10-02 15:26:11 -0700 |
|---|---|---|
| committer | Marcelo Vanzin <vanzin@cloudera.com> | 2015-10-02 15:26:11 -0700 |
| commit | f85aa06464a10f5d1563302fd76465dded475a12 (patch) | |
| tree | 3df828fb2774c9ff7182ccc2f67ca500df13c4ae /sbin/start-history-server.sh | |
| parent | b0baa11d3bb0e1e44553989b91a5efb9d04bc594 (diff) | |
| download | spark-f85aa06464a10f5d1563302fd76465dded475a12.tar.gz spark-f85aa06464a10f5d1563302fd76465dded475a12.tar.bz2 spark-f85aa06464a10f5d1563302fd76465dded475a12.zip | |
[SPARK-10317] [CORE] Compatibility between history server script and functionality
Compatibility between history server script and functionality
The history server has its argument parsing class in HistoryServerArguments. However, this doesn't get involved in the start-history-server.sh codepath where the $0 arg is assigned to spark.history.fs.logDirectory and all other arguments discarded (e.g --property-file.)
This stops the other options being usable from this script
Author: Joshi <rekhajoshm@gmail.com>
Author: Rekha Joshi <rekhajoshm@gmail.com>
Closes #8758 from rekhajoshm/SPARK-10317.
Diffstat (limited to 'sbin/start-history-server.sh')
| -rwxr-xr-x | sbin/start-history-server.sh | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/sbin/start-history-server.sh b/sbin/start-history-server.sh index 7172ad15d8..9034e5715c 100755 --- a/sbin/start-history-server.sh +++ b/sbin/start-history-server.sh @@ -30,10 +30,4 @@ sbin="`cd "$sbin"; pwd`" . "$sbin/spark-config.sh" . "$SPARK_PREFIX/bin/load-spark-env.sh" -if [ $# != 0 ]; then - echo "Using command line arguments for setting the log directory is deprecated. Please " - echo "set the spark.history.fs.logDirectory configuration option instead." - export SPARK_HISTORY_OPTS="$SPARK_HISTORY_OPTS -Dspark.history.fs.logDirectory=$1" -fi - -exec "$sbin"/spark-daemon.sh start org.apache.spark.deploy.history.HistoryServer 1 +exec "$sbin"/spark-daemon.sh start org.apache.spark.deploy.history.HistoryServer 1 $@ |