You are viewing a plain text version of this content. The canonical link for it is here.
Posted to user@giraph.apache.org by Claudio Martella <cl...@gmail.com> on 2011/09/29 16:55:54 UTC
Can't setClass in GiraphJob
Hello list,
I see I cannot submit a GiraphJob with my own Vertex as it doesn't
implement Vertex (it extends MutableVertex which extends BasicVertex)
but GiraphJob.setClass() calls:
getConfiguration().setClass(VERTEX_CLASS, vertexClass, Vertex.class);
which ensures my class implements Vertex which is not true. I think it
could be somehow connected to GIRAPH-36 somehow and the recent changes
to Vertex and BasicVertex.
I guess that should be BasicVertex.class, or?
--
Claudio Martella
claudio.martella@gmail.com
Re: Can't setClass in GiraphJob
Posted by Claudio Martella <cl...@gmail.com>.
No, the thing is deeper than just that fix. I'll have to wait.
Thanks
On Thu, Sep 29, 2011 at 5:05 PM, Claudio Martella
<cl...@gmail.com> wrote:
> i did "fix" it somehow, there are two or three lines related to this.
>
> I attach the diff.
>
> On Thu, Sep 29, 2011 at 5:01 PM, Jake Mannix <ja...@gmail.com> wrote:
>>
>> On Thu, Sep 29, 2011 at 7:55 AM, Claudio Martella
>> <cl...@gmail.com> wrote:
>>>
>>> Hello list,
>>>
>>> I see I cannot submit a GiraphJob with my own Vertex as it doesn't
>>> implement Vertex (it extends MutableVertex which extends BasicVertex)
>>> but GiraphJob.setClass() calls:
>>> getConfiguration().setClass(VERTEX_CLASS, vertexClass, Vertex.class);
>>>
>>> which ensures my class implements Vertex which is not true. I think it
>>> could be somehow connected to GIRAPH-36 somehow and the recent changes
>>> to Vertex and BasicVertex.
>>>
>>> I guess that should be BasicVertex.class, or?
>>
>> This is definitely part of GIRAPH-36, I ran into this too. You can try
>> changing that
>> one line, but I think there is more to it than that. I need to finish up
>> the work on my
>> patch for that, sorry it's been a little while, it needs to get fixed, you
>> are right (that's
>> why I set the priority on that ticket as "Blocker", to remind myself to get
>> it done!).
>>
>> -jake
>
>
>
> --
> Claudio Martella
> claudio.martella@gmail.com
>
--
Claudio Martella
claudio.martella@gmail.com
Re: Can't setClass in GiraphJob
Posted by Claudio Martella <cl...@gmail.com>.
i did "fix" it somehow, there are two or three lines related to this.
I attach the diff.
On Thu, Sep 29, 2011 at 5:01 PM, Jake Mannix <ja...@gmail.com> wrote:
>
> On Thu, Sep 29, 2011 at 7:55 AM, Claudio Martella
> <cl...@gmail.com> wrote:
>>
>> Hello list,
>>
>> I see I cannot submit a GiraphJob with my own Vertex as it doesn't
>> implement Vertex (it extends MutableVertex which extends BasicVertex)
>> but GiraphJob.setClass() calls:
>> getConfiguration().setClass(VERTEX_CLASS, vertexClass, Vertex.class);
>>
>> which ensures my class implements Vertex which is not true. I think it
>> could be somehow connected to GIRAPH-36 somehow and the recent changes
>> to Vertex and BasicVertex.
>>
>> I guess that should be BasicVertex.class, or?
>
> This is definitely part of GIRAPH-36, I ran into this too. You can try
> changing that
> one line, but I think there is more to it than that. I need to finish up
> the work on my
> patch for that, sorry it's been a little while, it needs to get fixed, you
> are right (that's
> why I set the priority on that ticket as "Blocker", to remind myself to get
> it done!).
>
> -jake
--
Claudio Martella
claudio.martella@gmail.com
Re: Can't setClass in GiraphJob
Posted by Jake Mannix <ja...@gmail.com>.
On Thu, Sep 29, 2011 at 7:55 AM, Claudio Martella <
claudio.martella@gmail.com> wrote:
> Hello list,
>
> I see I cannot submit a GiraphJob with my own Vertex as it doesn't
> implement Vertex (it extends MutableVertex which extends BasicVertex)
> but GiraphJob.setClass() calls:
> getConfiguration().setClass(VERTEX_CLASS, vertexClass, Vertex.class);
>
> which ensures my class implements Vertex which is not true. I think it
> could be somehow connected to GIRAPH-36 somehow and the recent changes
> to Vertex and BasicVertex.
>
> I guess that should be BasicVertex.class, or?
>
This is definitely part of GIRAPH-36, I ran into this too. You can try
changing that
one line, but I think there is more to it than that. I need to finish up
the work on my
patch for that, sorry it's been a little while, it needs to get fixed, you
are right (that's
why I set the priority on that ticket as "Blocker", to remind myself to get
it done!).
-jake