incubator-hama-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Edward J. Yoon (Updated) (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HAMA-469) The statusUpdate() method should be called finally.
Date Mon, 07 Nov 2011 11:03:51 GMT

     [ https://issues.apache.org/jira/browse/HAMA-469?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Edward J. Yoon updated HAMA-469:
--------------------------------

          Component/s: bsp
    Affects Version/s: 0.3.0
        Fix Version/s: 0.4.0
    
> The statusUpdate() method should be called finally.
> ---------------------------------------------------
>
>                 Key: HAMA-469
>                 URL: https://issues.apache.org/jira/browse/HAMA-469
>             Project: Hama
>          Issue Type: Bug
>          Components: bsp
>    Affects Versions: 0.3.0
>            Reporter: Edward J. Yoon
>            Priority: Minor
>             Fix For: 0.4.0
>
>
> {code}
>   public void sync() throws InterruptedException {
>     try {
>       enterBarrier();
>       Iterator<Entry<InetSocketAddress, LinkedList<BSPMessage>>> it
= messenger
>           .getMessageIterator();
>       while (it.hasNext()) {
>         Entry<InetSocketAddress, LinkedList<BSPMessage>> entry = it.next();
>         final InetSocketAddress addr = entry.getKey();
>         final Iterable<BSPMessage> messages = entry.getValue();
>         final BSPMessageBundle bundle = combineMessages(messages);
>         if (conf.getBoolean("bsp.checkpoint.enabled", false)) {
>           checkpoint(checkpointedPath(), bundle);
>         }
>         // remove this message during runtime to save a bit of memory
>         it.remove();
>         messenger.transfer(addr, bundle);
>       }
>       leaveBarrier();
>       currentTaskStatus.incrementSuperstepCount();
>       umbilical.statusUpdate(taskId, currentTaskStatus);
>       // Clear outgoing queues.
>       messenger.clearOutgoingQueues();
>     } catch (Exception e) {
>       LOG.fatal(
>           "Caught exception during superstep "
>               + currentTaskStatus.getSuperstepCount() + "!", e);
>     }
>   }
> {code}
> I think, the task status should be set to "Failed" when exception occurs in the sync()
method. And the statusUpdate() method should be called finally.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message