hama-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Tommaso Teofili <tommaso.teof...@gmail.com>
Subject Re: Combine all Writables into a new package
Date Mon, 21 Oct 2013 13:35:02 GMT
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
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message