Re: some giraph code

2012-01-06 Thread Sebastian Schelter
I have already been in email contact with Eugenia, because a colleague of mine is currently at UCI. I'll cc this mail to her :) --sebastian On 06.01.2012 14:20, Claudio Martella wrote: Hello guys, today I stumbled upon this:

[jira] [Updated] (GIRAPH-118) Clarify messages behavior in BasicVertex

2012-01-06 Thread Claudio Martella (Updated) (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-118?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Claudio Martella updated GIRAPH-118: Description: initialize() can receive a null parameter for messages (at least that's

[jira] [Commented] (GIRAPH-118) Clarify messages behavior in BasicVertex

2012-01-06 Thread Claudio Martella (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-118?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13181340#comment-13181340 ] Claudio Martella commented on GIRAPH-118: - As a user with some experience with

Re: some giraph code

2012-01-06 Thread Mattmann, Chris A (388J)
Hi Sebastian, Great. It might be good to either (a) copy giraph-dev@ on the discussion to keep the rest of the community outside of the specific sub-group you're talking to aware of whats up; or (b) send a summary of the discussion here. It's important to demonstrate that discussion within Apache

Re: some giraph code

2012-01-06 Thread Sebastian Schelter
@Chris I have to clarify this. I haven't been discussing Giraph-specific issues outside of the mailinglist, I just know Eugenia and know that she is working on porting some algorithms onto Pregel-like platforms. @Eugenia Sorry for the confusion :) We (the Giraph community) stumbled upon your

some weird code

2012-01-06 Thread Claudio Martella
Hello, I hope somebody can shed some light on a piece of code i'm looking at while working on GIRAPH-45 (and this code is also the object of GIRAPH-95, so we'd probably get two birds with one stone here). The code is taking care of vertex resolving in BasicRPCCommunication::prepareSuperstep():

[jira] [Created] (GIRAPH-119) VertexCombiner should work on IterableM instead of ListM

2012-01-06 Thread Claudio Martella (Created) (JIRA)
VertexCombiner should work on IterableM instead of ListM Key: GIRAPH-119 URL: https://issues.apache.org/jira/browse/GIRAPH-119 Project: Giraph Issue Type: Improvement

[jira] [Updated] (GIRAPH-119) VertexCombiner should work on IterableM instead of ListM

2012-01-06 Thread Claudio Martella (Updated) (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-119?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Claudio Martella updated GIRAPH-119: Attachment: GIRAPH-119.diff Trivial refactor to solve the issue.

[jira] [Commented] (GIRAPH-119) VertexCombiner should work on IterableM instead of ListM

2012-01-06 Thread Jakob Homan (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-119?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13181522#comment-13181522 ] Jakob Homan commented on GIRAPH-119: +1. The change log isn't usually modified as

Re: [jira] [Commented] (GIRAPH-119) VertexCombiner should work on IterableM instead of ListM

2012-01-06 Thread Claudio Martella
Ok, I didn't know. I usually do it as it requires 0-knowledge by a second hand. Btw, why would it conflict? On Fri, Jan 6, 2012 at 8:05 PM, Jakob Homan (Commented) (JIRA) j...@apache.org wrote:    [

Re: [jira] [Commented] (GIRAPH-119) VertexCombiner should work on IterableM instead of ListM

2012-01-06 Thread Jakob Homan
Maybe if changelog got modified (unlikely, I know). Doesn't hurt anything. Also, for non-committer contributors, they shouldn't modify the log since they don't know who may commit their code (ie bob via joe) On Fri, Jan 6, 2012 at 11:07 AM, Claudio Martella claudio.marte...@gmail.com wrote:

[jira] [Commented] (GIRAPH-119) VertexCombiner should work on IterableM instead of ListM

2012-01-06 Thread Hudson (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-119?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13181541#comment-13181541 ] Hudson commented on GIRAPH-119: --- Integrated in Giraph-trunk-Commit #61 (See

Re: some weird code

2012-01-06 Thread Avery Ching
Hi Claudio, answers inline: On 1/6/12 8:25 AM, Claudio Martella wrote: Hello, I hope somebody can shed some light on a piece of code i'm looking at while working on GIRAPH-45 (and this code is also the object of GIRAPH-95, so we'd probably get two birds with one stone here). The code is

[jira] [Commented] (GIRAPH-119) VertexCombiner should work on IterableM instead of ListM

2012-01-06 Thread Avery Ching (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-119?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13181566#comment-13181566 ] Avery Ching commented on GIRAPH-119: Minor nit @Override public

[jira] [Commented] (GIRAPH-118) Clarify messages behavior in BasicVertex

2012-01-06 Thread Avery Ching (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-118?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13181570#comment-13181570 ] Avery Ching commented on GIRAPH-118: Seems reasonable. Please make sure to update the

[jira] [Updated] (GIRAPH-118) Clarify messages behavior in BasicVertex

2012-01-06 Thread Claudio Martella (Updated) (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-118?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Claudio Martella updated GIRAPH-118: Attachment: GIRAPH-119.diff Apparently the initialize() issue is also true for other

[jira] [Updated] (GIRAPH-118) Clarify messages behavior in BasicVertex

2012-01-06 Thread Claudio Martella (Updated) (JIRA)
[ https://issues.apache.org/jira/browse/GIRAPH-118?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Claudio Martella updated GIRAPH-118: Attachment: GIRAPH-118.diff Messed up with issue number in patch filename, sorry :)