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

ASF GitHub Bot commented on HELIX-537:
--------------------------------------

Github user atcurtis commented on the pull request:

    https://github.com/apache/helix/pull/8#issuecomment-62669668
  
    Oops,
    
    I pushed the wrong commit to github. I’ll force push the correct one to my 
repo.
    
    On Nov 11, 2014, at 8:42 PM, Kanak Biscuitwala <[email protected]> 
wrote:
    
    > [INFO] -------------------------------------------------------------
    > [ERROR] COMPILATION ERROR :
    > [INFO] -------------------------------------------------------------
    > [ERROR] 
/Users/kanak/Developer/incubator-helix/helix-core/src/main/java/org/apache/helix/task/TaskStateModel.java:[70,2]
 missing return statement
    > [INFO] 1 error
    > [INFO] -------------------------------------------------------------
    > [INFO] 
------------------------------------------------------------------------
    > [INFO] BUILD FAILURE
    > 
    > TaskStateModel#shutdown has return type boolean, but returns nothing.
    > 
    > —
    > Reply to this email directly or view it on GitHub.
    > 


> org.apache.helix.task.TaskStateModel should have a shutdown method.
> -------------------------------------------------------------------
>
>                 Key: HELIX-537
>                 URL: https://issues.apache.org/jira/browse/HELIX-537
>             Project: Apache Helix
>          Issue Type: Bug
>          Components: helix-core
>    Affects Versions: 0.6.3
>            Reporter: Antony T Curtis
>            Priority: Blocker
>
> There should be a shutdown method to terminate the Timer and Executor which 
> the org.apache.helix.task.TaskStateModel class creates.
> ie.
> {noformat}
> public boolean shutdown(long timeout, TimeUnit unit)
>   throws InterruptedException
> {
>   reset();
>   _taskExecutor.shutdown();
>   _timer.cancel();
>   return _taskExecutor.awaitTermination(timeout, unit);
> }
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to