---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3349/#review4168
---
Had a quick look over your changes and everything looked good. I think
[
https://issues.apache.org/jira/browse/GIRAPH-116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13178348#comment-13178348
]
jirapos...@reviews.apache.org commented on GIRAPH-116:
--
Hi,
I'm working on an algorithm that computes a global value of the graph
(its so called effective diameter) and I have an Aggregator with which
this value can be computed. What would be the correct place to implement
this computation?
I thought about WorkerContext first, but it seems that this i
DefaultWorkerContext should preserve the method signatures of WorkerContext
---
Key: GIRAPH-117
URL: https://issues.apache.org/jira/browse/GIRAPH-117
Project: Giraph
Iss
[
https://issues.apache.org/jira/browse/GIRAPH-117?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sebastian Schelter updated GIRAPH-117:
--
Attachment: GIRAPH-117.patch
> DefaultWorkerContext should preserve the method sign
> On 2012-01-02 11:04:25, Sebastian Schelter wrote:
> > Had a quick look over your changes and everything looked good. I think it's
> > right to assume that most implementations will use static graphs and to
> > offer EdgeListVertex as the default extension point for this. The only
> > thing I
[
https://issues.apache.org/jira/browse/GIRAPH-116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13178498#comment-13178498
]
jirapos...@reviews.apache.org commented on GIRAPH-116:
--
bq. On 201
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3349/#review4170
---
Ship it!
Ok, I understood this wrong. I'm fine with the changes then.
[
https://issues.apache.org/jira/browse/GIRAPH-116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13178501#comment-13178501
]
jirapos...@reviews.apache.org commented on GIRAPH-116:
--
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3349/
---
(Updated 2012-01-02 18:51:05.007864)
Review request for giraph.
Changes
--
[
https://issues.apache.org/jira/browse/GIRAPH-116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13178506#comment-13178506
]
jirapos...@reviews.apache.org commented on GIRAPH-116:
--
[
https://issues.apache.org/jira/browse/GIRAPH-117?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13178510#comment-13178510
]
Avery Ching commented on GIRAPH-117:
+1, looks fine to me.
> DefaultW
[
https://issues.apache.org/jira/browse/GIRAPH-116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13178514#comment-13178514
]
Hudson commented on GIRAPH-116:
---
Integrated in Giraph-trunk-Commit #60 (See
[https://builds
[
https://issues.apache.org/jira/browse/GIRAPH-116?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Avery Ching resolved GIRAPH-116.
Resolution: Fixed
Thanks for the fast review Sebastian. Hopefully new folks will be less
irritate
There is support that Claudio added for exporting aggregator values
(https://issues.apache.org/jira/browse/GIRAPH-10). You could probably
also do your final computation there if that is what you want (if you
want to do end of application manipulation). WorkerContext would be a
good way to do
[
https://issues.apache.org/jira/browse/GIRAPH-93?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13178607#comment-13178607
]
Dmitriy V. Ryaboy commented on GIRAPH-93:
-
I suppose one could fake this by adding
[
https://issues.apache.org/jira/browse/GIRAPH-93?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13178616#comment-13178616
]
Avery Ching commented on GIRAPH-93:
---
Interesting idea. Maybe I'll look around for an exa
17 matches
Mail list logo
