hama-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chia-Hung Lin <cli...@googlemail.com>
Subject Re: Combine all Writables into a new package
Date Mon, 21 Oct 2013 16:31:11 GMT
+1 for hama-io or hama-commons


On 21 October 2013 21:35, Tommaso Teofili <tommaso.teofili@gmail.com> wrote:
> what about creating a module for that (Writables and InputFormats for now)
> hama-io / hama-commons that can be used by both (containing math stuff as
> well) ?
>
> Tommaso
>
>
> 2013/10/21 Martin Illecker <millecker@apache.org>
>
>> VectorWritable and MatrixWritable have both some dependencies
>> to org.apache.hama.ml.math.
>> (DenseDoubleVector, DoubleVector and DenseDoubleMatrix, DoubleMatrix)
>>
>> If we move VectorWritable and MatrixWritable to core (e.g.,
>> org.apache.hama.io.writable), we have to move org.apache.hama.ml.math as
>> well.
>> I think that's not possible because of other classes in hama-ml depending
>> on ml.math.
>>
>> Temporary I will have to copy VectorWritable to the core to use it in a
>> test case.
>>
>>
>> 2013/10/21 Tommaso Teofili <tommaso.teofili@gmail.com>
>>
>> > 2013/10/21 Martin Illecker <millecker@apache.org>
>> >
>> > > Hello,
>> > >
>> > > regarding to my Hama Pipes test case [1], I want to use VectorWritable
>> > > inside the hama-core module.
>> > > Therefore I would need a dependency to hama-ml but this will cause a
>> > cyclic
>> > > dependency.
>> > >
>> > > So is it possible to move both writables, VectorWritable and
>> > > MatrixWritable, from org.apache.hama.ml.writable into a new package?
>> > > e.g., org.apache.hama.io.writable based on [2]
>> > >
>> >
>> > I think this really makes sense.
>> >
>> >
>> > >
>> > > Regarding to [3] we can also move TextArrayWritable
>> > > from org.apache.hama.bsp into this new package.
>> > >
>> > > Do you think we can move the writables of org.apache.hama.ml.writable
>> to
>> > > the core module?
>> > >
>> >
>> > +1
>> >
>> >
>> > > And can we do the package refactoring [2] of org.apache.hama.bsp
>> > submitted
>> > > by Suraj?
>> > >
>> >
>> > +1 here too.
>> >
>> > Tommaso
>> >
>> >
>> > >
>> > > Thanks!
>> > >
>> > > Martin
>> > >
>> > > [1] https://issues.apache.org/jira/browse/HAMA-808
>> > > [2]
>> > https://issues.apache.org/jira/secure/attachment/12609417/bsplist.txt
>> > > [3] https://issues.apache.org/jira/browse/HAMA-727
>> > >
>> >
>>

Mime
View raw message