[ 
https://issues.apache.org/jira/browse/YARN-7601?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16303705#comment-16303705
 ] 

Sampada Dehankar edited comment on YARN-7601 at 12/26/17 9:47 AM:
------------------------------------------------------------------

[~asuresh]: Can you please review this patch? Looks like this particular test 
[TestContainerSchedulerQueuing: testKillOnlyRequiredOpportunisticContainers] 
passes intermittently. After submitting the same patch again, it passed.


was (Author: sampada15):
[~asuresh]: Can you please review this patch?

> Incorrect container states recovered as LevelDB uses alphabetical order
> -----------------------------------------------------------------------
>
>                 Key: YARN-7601
>                 URL: https://issues.apache.org/jira/browse/YARN-7601
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Sampada Dehankar
>            Assignee: Sampada Dehankar
>         Attachments: YARN-7601.001.patch, YARN-7601.002.patch
>
>
> LevelDB stores key-value pairs in the alphabetical order. Container id 
> concatenated by its state is used as key. So, even if container goes through 
> any states in its life cycle, the order of states for following values 
> retrieved from LevelDB is always going to be as below`:
> LAUNCHED
> PAUSED
> QUEUED
> For eg: If a container is LAUNCHED then PAUSED and LAUNCHED again, the 
> recovered container state is PAUSED currently instead of LAUNCHED.
> We propose to store the timestamp as the value while making call to 
>   
>   storeContainerLaunched
>   storeContainerPaused
>   storeContainerQueued
>   
> so that correct container state is recovered based on timestamps.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to