Re: Review Request: aggregator at input super step

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11525/ --- (Updated June 12, 2013, 3:25 a.m.) Review request for giraph. Changes ---

[jira] [Commented] (GIRAPH-623) Website Documentation: table of parameters

[ https://issues.apache.org/jira/browse/GIRAPH-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13680807#comment-13680807 ] Armando Miraglia commented on GIRAPH-623: - Hi, so I have managed to do something.

Re: Review Request: Jython for Computation

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11709/ --- (Updated June 11, 2013, 7:31 p.m.) Review request for giraph. Summary (update

Re: Review Request: review updates

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11709/ --- (Updated June 11, 2013, 5:29 p.m.) Review request for giraph. Changes ---

Re: Review Request: Jython for Giraph

> On June 11, 2013, 12:07 a.m., Avery Ching wrote: > > giraph-core/src/main/java/org/apache/giraph/jython/JythonComputationFactory.java, > > line 57 > > > > > > Wrapped LOG.info please. > > Nitay Joffe wrote: > Fi

Re: Review Request: review updates

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11709/ --- (Updated June 11, 2013, 5:28 p.m.) Review request for giraph. Summary (update

Re: Review Request: Jython for Giraph

> On June 11, 2013, 5:13 p.m., Alessandro Presta wrote: > > Pretty cool! It will be really useful once there is better support for > > types: > > 1) some kind of type inference/casting/automatic serialization, so that one > > can write setValue(4) instead of setValue(IntWritable(4)) > > 2) poss

Re: Review Request: Jython for Giraph

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11709/#review21705 --- Pretty cool! It will be really useful once there is better support f

Re: Review Request: Jython for Giraph

> On June 11, 2013, 12:07 a.m., Avery Ching wrote: > > giraph-core/src/main/java/org/apache/giraph/conf/AbstractConfOption.java, > > lines 49-66 > > > > > > my personal preference here is for adding boolean exists(Conf

Re: Review Request: Jython for Giraph

> On June 11, 2013, 5:51 a.m., Nitay Joffe wrote: > > Regarding your comment - yeah I'll add it to GiraphRunner. Right now you > > have to specify at the very least the types IVEMM, the python script path > > (as resource in jar) and the computation class name. Any thoughts how to > > make thi

[jira] [Commented] (GIRAPH-676) A short tutorial on getting started with Giraph

[ https://issues.apache.org/jira/browse/GIRAPH-676?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13680382#comment-13680382 ] Claudio Martella commented on GIRAPH-676: - I think moving it at the end with a spe

[jira] [Commented] (GIRAPH-676) A short tutorial on getting started with Giraph

[ https://issues.apache.org/jira/browse/GIRAPH-676?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13680274#comment-13680274 ] Yazan Boshmaf commented on GIRAPH-676: -- Claudio, Avery: I see your point =) I will

[jira] [Commented] (GIRAPH-676) A short tutorial on getting started with Giraph

[ https://issues.apache.org/jira/browse/GIRAPH-676?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13680270#comment-13680270 ] Avery Ching commented on GIRAPH-676: +1 to what Claudio said. I think it might be sim

[jira] [Commented] (GIRAPH-676) A short tutorial on getting started with Giraph

[ https://issues.apache.org/jira/browse/GIRAPH-676?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13680267#comment-13680267 ] Claudio Martella commented on GIRAPH-676: - You have good points. My point is that