答复: bug in Worker.scala, ExecutorRunner is not serializable

2015-09-17 Thread Huangguowei
Not error in normal case. But if I want to ask Worker through akkaUrl to get executors status, it will cause Exception. 发件人: Sean Owen [mailto:so...@cloudera.com] 发送时间: 2015年9月17日 15:54 收件人: Huangguowei; Dev 主题: Re: bug in Worker.scala, ExecutorRunner is not serializable Did this cause an err

答复: bug in Worker.scala, ExecutorRunner is not serializable

2015-09-17 Thread Huangguowei
Thanks for your reply. I just want to do some monitors, never mind! 发件人: Shixiong Zhu [mailto:zsxw...@gmail.com] 发送时间: 2015年9月17日 17:23 收件人: Huangguowei; dev@spark.apache.org 主题: Re: bug in Worker.scala, ExecutorRunner is not serializable RequestWorkerState is an internal message between Worker a

Re: 答复: bug in Worker.scala, ExecutorRunner is not serializable

2015-09-18 Thread Reynold Xin
Maybe we should add some inline comment explaining why it is ok for that message to be not serializable. On Thu, Sep 17, 2015 at 4:08 AM, Huangguowei wrote: > Thanks for your reply. I just want to do some monitors, never mind! > > > > *发件人:* Shixiong Zhu [mailto:zsxw...@gmail.com] > *发送时间:* 201

Re: 答复: bug in Worker.scala, ExecutorRunner is not serializable

2015-09-18 Thread Shixiong Zhu
I'm wondering if we should create a tag trait (e.g., LocalMessage) for messages like this and add the comment in the trait. Looks better than adding inline comments for all these messages. Best Regards, Shixiong Zhu 2015-09-18 15:10 GMT+08:00 Reynold Xin : > Maybe we should add some inline comme

Re: 答复: bug in Worker.scala, ExecutorRunner is not serializable

2015-09-18 Thread Reynold Xin
Sounds good. On Fri, Sep 18, 2015 at 8:50 AM, Shixiong Zhu wrote: > I'm wondering if we should create a tag trait (e.g., LocalMessage) for > messages like this and add the comment in the trait. Looks better than > adding inline comments for all these messages. > > Best Regards, > Shixiong Zhu >