Yeah, true. It would be good to add it back. Matei
On Nov 5, 2013, at 6:40 PM, Aaron Davidson <aa...@databricks.com> wrote: > Looks like the appId thing was removed accidentally quite a while ago: > https://github.com/apache/incubator-spark/commit/85a35c68401e171df0b72b172a689d8c4e412199 > and has gone unnoticed since. I have no objections to adding it back in... > > > On Tue, Nov 5, 2013 at 2:36 PM, Reynold Xin <r...@apache.org> wrote: > >> +aaron on this one since he changed the executor runner. (I think it is >> probably an oversight but Aaron should confirm.) >> >> >> >> >> On Tue, Nov 5, 2013 at 10:44 AM, Imran Rashid <im...@quantifind.com>wrote: >> >>> Hi, >>> >>> a while back, ExecutorRunner was changed so the command line args included >>> the appId. >>> >>> https://github.com/mesos/spark/pull/467 >>> >>> Those changes seem to be gone from the latest code. Was that intentional, >>> or just an oversight? I'll add it back in if it was removed accidentally, >>> but wanted to check in case there is some reason it shouldn't be there. >>> >>> thanks, >>> Imran >>> >> >>
