[jira] [Commented] (GIRAPH-27) Mutable static global state in Vertex.java should be refactored

[ https://issues.apache.org/jira/browse/GIRAPH-27?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13100901#comment-13100901 ] Jake Mannix commented on GIRAPH-27: --- Awesome, thanks Avery. Looks good to me. In

[jira] [Created] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

Introduce new primitive-specific MutableVertex subclasses - Key: GIRAPH-28 URL: https://issues.apache.org/jira/browse/GIRAPH-28 Project: Giraph Issue Type: New Feature

[jira] [Commented] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13101021#comment-13101021 ] Jake Mannix commented on GIRAPH-28: --- This is a toy version of

[jira] [Commented] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13102430#comment-13102430 ] Jake Mannix commented on GIRAPH-28: --- So Avery, the question I have for you is regarding

[jira] [Commented] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13102442#comment-13102442 ] Jake Mannix commented on GIRAPH-28: --- I like the Iterator more than ImmutableList, yeah,

[jira] [Commented] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=1310#comment-1310 ] Jake Mannix commented on GIRAPH-28: --- Ok, so I went ahead and made a 'straw-man'

[jira] [Created] (GIRAPH-31) Hide the SortedMapI, EdgeI,E in Vertex from client visibility (impl. detail), replace with appropriate accessor methods

Hide the SortedMapI, EdgeI,E in Vertex from client visibility (impl. detail), replace with appropriate accessor methods --- Key: GIRAPH-31

[jira] [Commented] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13103339#comment-13103339 ] Jake Mannix commented on GIRAPH-28: --- I'm suggesting that iterator() be always sorted.

[jira] [Commented] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13103341#comment-13103341 ] Jake Mannix commented on GIRAPH-28: --- Also, to contradict my 1st and 3rd points, Dmitriy

[jira] [Commented] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13103357#comment-13103357 ] Jake Mannix commented on GIRAPH-28: --- The alternative to IterableEdgeI, E is IterableI,

[jira] [Commented] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13103363#comment-13103363 ] Jake Mannix commented on GIRAPH-28: --- As for sorting, I'd imagine that assuming it always

[jira] [Commented] (GIRAPH-31) Hide the SortedMapI, EdgeI,E in Vertex from client visibility (impl. detail), replace with appropriate accessor methods

[ https://issues.apache.org/jira/browse/GIRAPH-31?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13103745#comment-13103745 ] Jake Mannix commented on GIRAPH-31: --- And for the implementations which have both the

[jira] [Commented] (GIRAPH-31) Hide the SortedMapI, EdgeI,E in Vertex from client visibility (impl. detail), replace with appropriate accessor methods

[ https://issues.apache.org/jira/browse/GIRAPH-31?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13103798#comment-13103798 ] Jake Mannix commented on GIRAPH-31: --- +1 to that, given your argument on the current use

[jira] [Updated] (GIRAPH-31) Hide the SortedMapI, EdgeI,E in Vertex from client visibility (impl. detail), replace with appropriate accessor methods

[ https://issues.apache.org/jira/browse/GIRAPH-31?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jake Mannix updated GIRAPH-31: -- Attachment: GIRAPH-31.diff Updated patch - remove isSorted(), document the fact that the iterator may

[jira] [Commented] (GIRAPH-31) Hide the SortedMapI, EdgeI,E in Vertex from client visibility (impl. detail), replace with appropriate accessor methods

[ https://issues.apache.org/jira/browse/GIRAPH-31?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13103948#comment-13103948 ] Jake Mannix commented on GIRAPH-31: --- Sounds good to me! Lazy consensus is pretty common

[jira] [Commented] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13105004#comment-13105004 ] Jake Mannix commented on GIRAPH-28: --- Ok another patch coming soon for this, but good

[jira] [Commented] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13105568#comment-13105568 ] Jake Mannix commented on GIRAPH-28: --- I don't know what it was, I just re-patched with

[jira] [Updated] (GIRAPH-28) Introduce new primitive-specific MutableVertex subclasses

[ https://issues.apache.org/jira/browse/GIRAPH-28?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jake Mannix updated GIRAPH-28: -- Attachment: GIRAPH-28.diff Newly regenerated against trunk. Introduce new primitive-specific

[jira] [Commented] (GIRAPH-34) Failure of Vertex reflection for putVertexList from GIRAPH-27

[ https://issues.apache.org/jira/browse/GIRAPH-34?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13106717#comment-13106717 ] Jake Mannix commented on GIRAPH-34: --- Wait, why would the sending Vertex modify the

[jira] [Commented] (GIRAPH-34) Failure of Vertex reflection for putVertexList from GIRAPH-27

[ https://issues.apache.org/jira/browse/GIRAPH-34?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13106748#comment-13106748 ] Jake Mannix commented on GIRAPH-34: --- I'll definitely open another JIRA for the Vertex

[jira] [Commented] (GIRAPH-34) Failure of Vertex reflection for putVertexList from GIRAPH-27

[ https://issues.apache.org/jira/browse/GIRAPH-34?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13106780#comment-13106780 ] Jake Mannix commented on GIRAPH-34: --- Yeah, how do you do that, Dmitriy? Failure of

[jira] [Commented] (GIRAPH-34) Failure of Vertex reflection for putVertexList from GIRAPH-27

[ https://issues.apache.org/jira/browse/GIRAPH-34?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13106804#comment-13106804 ] Jake Mannix commented on GIRAPH-34: --- +1 from me - although I haven't run it on an actual

[jira] [Commented] (GIRAPH-12) Investigate communication improvements

[ https://issues.apache.org/jira/browse/GIRAPH-12?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13106842#comment-13106842 ] Jake Mannix commented on GIRAPH-12: --- Hey Hyunsik, if you're going to write a benchmark

[jira] [Created] (GIRAPH-36) Ensure that subclassing BasicVertex is possible by user apps

Ensure that subclassing BasicVertex is possible by user apps Key: GIRAPH-36 URL: https://issues.apache.org/jira/browse/GIRAPH-36 Project: Giraph Issue Type: Improvement

[jira] [Commented] (GIRAPH-37) Implement Netty-backed rpc solution

[ https://issues.apache.org/jira/browse/GIRAPH-37?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13107165#comment-13107165 ] Jake Mannix commented on GIRAPH-37: --- We should make sure we don't all work on the same

[jira] [Assigned] (GIRAPH-12) Investigate communication improvements

[ https://issues.apache.org/jira/browse/GIRAPH-12?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jake Mannix reassigned GIRAPH-12: - Assignee: Avery Ching (was: Hyunsik Choi) Investigate communication improvements

[jira] [Assigned] (GIRAPH-12) Investigate communication improvements

[ https://issues.apache.org/jira/browse/GIRAPH-12?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jake Mannix reassigned GIRAPH-12: - Assignee: Hyunsik Choi (was: Avery Ching) Sorry, my 4-year old clicked when I was looking at

[jira] [Commented] (GIRAPH-36) Ensure that subclassing BasicVertex is possible by user apps

[ https://issues.apache.org/jira/browse/GIRAPH-36?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13107281#comment-13107281 ] Jake Mannix commented on GIRAPH-36: --- Initial thoughts: VertexReader defines a

[jira] [Commented] (GIRAPH-36) Ensure that subclassing BasicVertex is possible by user apps

[ https://issues.apache.org/jira/browse/GIRAPH-36?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13107282#comment-13107282 ] Jake Mannix commented on GIRAPH-36: --- In fact, thinking about VertexReader further, it