diff options
| author | CodingCat <zhunansjtu@gmail.com> | 2014-02-22 20:21:15 -0800 |
|---|---|---|
| committer | Shivaram Venkataraman <shivaram@eecs.berkeley.edu> | 2014-02-22 20:21:15 -0800 |
| commit | 437b62fcb03cc84c6581eb667ddbddd817f7f9f0 (patch) | |
| tree | f3fb5bb510d8f1a71f04976aef1a153bff5867c1 | |
| parent | 29ac7ea52fbb0c6531e14305e2fb1ccba9678f7e (diff) | |
| download | spark-437b62fcb03cc84c6581eb667ddbddd817f7f9f0.tar.gz spark-437b62fcb03cc84c6581eb667ddbddd817f7f9f0.tar.bz2 spark-437b62fcb03cc84c6581eb667ddbddd817f7f9f0.zip | |
[SPARK-1041] remove dead code in start script, remind user to set that in spark-env.sh
the lines in start-master.sh and start-slave.sh no longer work
in ec2, the host name has changed, e.g.
ubuntu@ip-172-31-36-93:~$ hostname
ip-172-31-36-93
also, the URL to fetch public DNS name also changed, e.g.
ubuntu@ip-172-31-36-93:~$ wget -q -O - http://instance-data.ec2.internal/latest/meta-data/public-hostname
ubuntu@ip-172-31-36-93:~$ (returns nothing)
since we have spark-ec2 project, we don't need to have such ec2-specific lines here, instead, user only need to set in spark-env.sh
Author: CodingCat <zhunansjtu@gmail.com>
Closes #588 from CodingCat/deadcode_in_sbin and squashes the following commits:
e4236e0 [CodingCat] remove dead code in start script, remind user set that in spark-env.sh
| -rwxr-xr-x | conf/spark-env.sh.template | 1 | ||||
| -rwxr-xr-x | sbin/start-master.sh | 9 | ||||
| -rwxr-xr-x | sbin/start-slave.sh | 9 |
3 files changed, 1 insertions, 18 deletions
diff --git a/conf/spark-env.sh.template b/conf/spark-env.sh.template index e2071e2ade..6432a56608 100755 --- a/conf/spark-env.sh.template +++ b/conf/spark-env.sh.template @@ -19,3 +19,4 @@ # - SPARK_WORKER_PORT / SPARK_WORKER_WEBUI_PORT # - SPARK_WORKER_INSTANCES, to set the number of worker processes per node # - SPARK_WORKER_DIR, to set the working directory of worker processes +# - SPARK_PUBLIC_DNS, to set the public dns name of the master diff --git a/sbin/start-master.sh b/sbin/start-master.sh index 3dcf7cc348..ec3dfdb419 100755 --- a/sbin/start-master.sh +++ b/sbin/start-master.sh @@ -40,13 +40,4 @@ if [ "$SPARK_MASTER_WEBUI_PORT" = "" ]; then SPARK_MASTER_WEBUI_PORT=8080 fi -# Set SPARK_PUBLIC_DNS so the master report the correct webUI address to the slaves -if [ "$SPARK_PUBLIC_DNS" = "" ]; then - # If we appear to be running on EC2, use the public address by default: - # NOTE: ec2-metadata is installed on Amazon Linux AMI. Check based on that and hostname - if command -v ec2-metadata > /dev/null || [[ `hostname` == *ec2.internal ]]; then - export SPARK_PUBLIC_DNS=`wget -q -O - http://instance-data.ec2.internal/latest/meta-data/public-hostname` - fi -fi - "$sbin"/spark-daemon.sh start org.apache.spark.deploy.master.Master 1 --ip $SPARK_MASTER_IP --port $SPARK_MASTER_PORT --webui-port $SPARK_MASTER_WEBUI_PORT diff --git a/sbin/start-slave.sh b/sbin/start-slave.sh index 524be38c62..b563400dc2 100755 --- a/sbin/start-slave.sh +++ b/sbin/start-slave.sh @@ -23,13 +23,4 @@ sbin=`dirname "$0"` sbin=`cd "$sbin"; pwd` -# Set SPARK_PUBLIC_DNS so slaves can be linked in master web UI -if [ "$SPARK_PUBLIC_DNS" = "" ]; then - # If we appear to be running on EC2, use the public address by default: - # NOTE: ec2-metadata is installed on Amazon Linux AMI. Check based on that and hostname - if command -v ec2-metadata > /dev/null || [[ `hostname` == *ec2.internal ]]; then - export SPARK_PUBLIC_DNS=`wget -q -O - http://instance-data.ec2.internal/latest/meta-data/public-hostname` - fi -fi - "$sbin"/spark-daemon.sh start org.apache.spark.deploy.worker.Worker "$@" |