[ https://issues.apache.org/jira/browse/GIRAPH-132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Avery Ching resolved GIRAPH-132. -------------------------------- Resolution: Not A Problem Thanks Eli, this was resolved in the massive checkstyle fix (GIRAPH-40). > Simplify boolean expression in GraphMapper::map() > ------------------------------------------------- > > Key: GIRAPH-132 > URL: https://issues.apache.org/jira/browse/GIRAPH-132 > Project: Giraph > Issue Type: Improvement > Reporter: Jakob Homan > Priority: Trivial > Labels: newbie > > The boolean expression in: > {noformat} @Override > public void map(Object key, Object value, Context context) > throws IOException, InterruptedException { > // map() only does computation > // 1) Run checkpoint per frequency policy. > // 2) For every vertex on this mapper, run the compute() function > // 3) Wait until all messaging is done. > // 4) Check if all vertices are done. If not goto 2). > // 5) Dump output. > if (done == true) { > return; > }{noformat} > can be simplified. -- 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