I just committed the fix: https://reviews.apache.org/r/10381/

Let me know if you're still seeing issues.


On Tue, Apr 9, 2013 at 11:09 AM, Benjamin Mahler
<[email protected]>wrote:

> Thanks for the report!
>
> The bug here is that we set the source for each task:
>
>         TaskInfo task;
>         task.set_name("Task " + lexical_cast<string>(taskId));
>         task.mutable_task_id()->set_value(lexical_cast<string>(taskId));
>         task.mutable_slave_id()->MergeFrom(offer.slave_id());
>         task.mutable_executor()->MergeFrom(executor);
>         *task.mutable_executor()->set_source("task_" + stringify(taskId));
> *
>
> I'll have a review out shortly to fix this.
>
>
> On Tue, Apr 9, 2013 at 2:43 AM, 王国栋 <[email protected]> wrote:
>
>> hi
>>
>> I am trying to run long-lived-framework in the trunk.
>> But I find the following error after task 1 finished.
>>
>> W0409 17:18:03.841472 15305 master.cpp:1566] Error validating task 1 :
>> Task
>> has invalid ExecutorInfo (existing ExecutorInfo with same ExecutorID is
>> not
>> compatible)
>>
>> Then all the tasks will be lost.
>>
>> The log of the framework is as follow:
>> Registered!
>> .Starting task 0 on guodong-Vostro-3400
>> Task 0 is in state 1
>> Task 0 is in state 2
>> .Starting task 1 on guodong-Vostro-3400
>> Task 1 is in state 5
>> .Starting task 2 on guodong-Vostro-3400
>> Task 2 is in state 5
>> .Starting task 3 on guodong-Vostro-3400
>> Task 3 is in state 5
>>
>>
>> I also go through the code of LongLivedFramework. And I can not understand
>> this error, since the ExecutorInfo is passed as the constructor arguments
>> of Scheduler.
>>
>> Best regards.
>>
>> Guodong
>>
>
>

Reply via email to