[ https://issues.apache.org/jira/browse/HDFS-11264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16084152#comment-16084152 ]
Rakesh R edited comment on HDFS-11264 at 7/12/17 3:35 PM: ---------------------------------------------------------- Attached new patch. Here, I have changed the order of Mover status check in SPS. With this change, we can avoid the unnecessary instantiation and starting of SPS thread if Mover is running. Also, avoids shutdown calls in the exception block if the thread is interrupted by the user or admin. [~umamaheswararao], please review. Thanks! +Note:+ I had re-based the branch code on latest trunk code, yesterday. In this patch, I have included few EC related modifications to make the test case run according to the trunk code. was (Author: rakeshr): Attached new patch. Here, I have changed the order of Mover status check in SPS. With this change, we can avoid the unnecessary instantiation and starting of SPS thread if Mover is running. Also, avoids shutdown calls in the exception block if the thread is interrupted by the user or admin. [~umamaheswararao], please review. Thanks! > [SPS]: Double checks to ensure that SPS/Mover are not running together > ---------------------------------------------------------------------- > > Key: HDFS-11264 > URL: https://issues.apache.org/jira/browse/HDFS-11264 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: datanode, namenode > Reporter: Wei Zhou > Assignee: Rakesh R > Attachments: HDFS-11264-HDFS-10285-01.patch, > HDFS-11264-HDFS-10285-02.patch, HDFS-11264-HDFS-10285-03.patch > > > As discussed in HDFS-10885, double checks needed to insure SPS/Mover not > running together, otherwise it may cause some issues. -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org