[ https://issues.apache.org/jira/browse/GIRAPH-90?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13151532#comment-13151532 ]
Jake Mannix commented on GIRAPH-90: ----------------------------------- It's probably not what we want, no. LongWritables are cheap, let's make new ones and return those. > LongDoubleFloatDoubleVertex has possibily the iterator() implementation broken > ------------------------------------------------------------------------------ > > Key: GIRAPH-90 > URL: https://issues.apache.org/jira/browse/GIRAPH-90 > Project: Giraph > Issue Type: Bug > Components: graph > Affects Versions: 0.70.0 > Reporter: Claudio Martella > Assignee: Claudio Martella > Fix For: 0.70.0 > > > iterator() implementation returns LongWritable which is cached in a final > variable and set() with the new value at next(). This could be misleading as > the user might create a list from the iterator's data. Something similar is > happening in the getMsgList() as well. > Is this really what we want? -- 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