[
https://issues.apache.org/jira/browse/HAMA-563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13260239#comment-13260239
]
Thomas Jungblut commented on HAMA-563:
--------------------------------------
Thanks, patch looks good to me, what do you think about making the checkpoint
interval methods package visible?
They are not documented and don't have a real usecase currently.
> Mods/fixes and refactoring for 0.5 release
> ------------------------------------------
>
> Key: HAMA-563
> URL: https://issues.apache.org/jira/browse/HAMA-563
> Project: Hama
> Issue Type: Bug
> Components: bsp
> Affects Versions: 0.4.0
> Reporter: Edward J. Yoon
> Assignee: Edward J. Yoon
> Priority: Blocker
> Fix For: 0.5.0
>
> Attachments: HAMA-563.patch
>
>
> {code}
> I have a few notes:
> There is a method called setMaxIteration in BSPJob, this should belong to
> the GraphJob.
> And there are a few methods that are not documented properly,
> namely setCheckPointInterval and setCheckPointFlag (we should make it
> package visible first until we resolved all the stuff arround fault
> tolerance).
> We shouldn't name the package dist and source, better would be bin and src.
> {code}
--
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