You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by pw...@apache.org on 2013/12/17 00:23:35 UTC

[2/2] git commit: Merge pull request #271 from ewencp/really-force-ssh-pseudo-tty-0.8

Merge pull request #271 from ewencp/really-force-ssh-pseudo-tty-0.8

Force pseudo-tty allocation in spark-ec2 script.

ssh commands need the -t argument repeated twice if there is no local
tty, e.g. if the process running spark-ec2 uses nohup and the parent
process exits.

Without this change, if you run the script this way (e.g. using nohup from a cron job), it will fail setting up the nodes because some of the ssh commands complain about missing ttys and then fail.

(This version is for the 0.8 branch. I've filed a separate request for master since changes to the script caused the patches to be different.)


Project: http://git-wip-us.apache.org/repos/asf/incubator-spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-spark/commit/f898238a
Tree: http://git-wip-us.apache.org/repos/asf/incubator-spark/tree/f898238a
Diff: http://git-wip-us.apache.org/repos/asf/incubator-spark/diff/f898238a

Branch: refs/heads/branch-0.8
Commit: f898238a26d2f16796dda51212baf3f267747783
Parents: 8f56390 2e2ead4
Author: Patrick Wendell <pw...@gmail.com>
Authored: Mon Dec 16 15:23:25 2013 -0800
Committer: Patrick Wendell <pw...@gmail.com>
Committed: Mon Dec 16 15:23:25 2013 -0800

----------------------------------------------------------------------
 ec2/spark_ec2.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------