Merge pull request #148 from squito/include_appId Include appId in executor cmd line args
add the appId back into the executor cmd line args. I also made a pretty lame regression test, just to make sure it doesn't get dropped in the future. not sure it will run on the build server, though, b/c `ExecutorRunner.buildCommandSeq()` expects to be abel to run the scripts in `bin`. Project: http://git-wip-us.apache.org/repos/asf/incubator-spark/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-spark/commit/3d4ad84b Tree: http://git-wip-us.apache.org/repos/asf/incubator-spark/tree/3d4ad84b Diff: http://git-wip-us.apache.org/repos/asf/incubator-spark/diff/3d4ad84b Branch: refs/heads/master Commit: 3d4ad84b63e440fd3f4b3edb1b120ff7c14a42d1 Parents: be7e8da ca66f5d Author: Reynold Xin <[email protected]> Authored: Thu Nov 7 11:08:27 2013 -0800 Committer: Reynold Xin <[email protected]> Committed: Thu Nov 7 11:08:27 2013 -0800 ---------------------------------------------------------------------- .../spark/deploy/worker/ExecutorRunner.scala | 2 +- .../executor/CoarseGrainedExecutorBackend.scala | 2 +- .../deploy/worker/ExecutorRunnerTest.scala | 20 ++++++++++++++++++++ 3 files changed, 22 insertions(+), 2 deletions(-) ----------------------------------------------------------------------
