reef-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mariia Mykhailova (JIRA)" <>
Subject [jira] [Resolved] (REEF-1692) Revert the ignorance for extra Evaluators requested
Date Fri, 20 Jan 2017 20:13:26 GMT


Mariia Mykhailova resolved REEF-1692.
       Resolution: Fixed
    Fix Version/s: 0.16

Resolved via [PR 1237|]

> Revert the ignorance for extra Evaluators requested
> ---------------------------------------------------
>                 Key: REEF-1692
>                 URL:
>             Project: REEF
>          Issue Type: Bug
>            Reporter: Julia
>            Assignee: Julia
>            Priority: Minor
>              Labels: FT
>             Fix For: 0.16
> The reason we did REEF-1688 for ignoring extra AllocatedEvaluators was because we thought
Yarn might give us additional Evalautors. In fact, it is a bug in REEF code. REEF-1691 has
been created to fix it. 
> The side effect for ignoring extra IAllocatedEvaluator are:
> * It hides potential bugs
> * Even we choose to ignore the extra IAllocatedEvaluator in EvaluatorManager, this event
can be received in other system state rather than WaitingForEvalautor. This is because as
soon as we get all the needed Evaluators, we change the system state and start to submit tasks,
and the system state can be SubmittingTasks, or ShuttingDown after it. By the time we receive
the extra IAllocatedEvaluator event, it is in                    unexpected state, IMRUDriver
will throw exception in those cases. 
> I would think we should revert the change in REEF-1688 to ensure the integrity of the

This message was sent by Atlassian JIRA

View raw message