Mailing-List: contact dev-help@hama.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hama.apache.org
Received-SPF: pass (nike.apache.org: domain of menonsuraj5@gmail.com
 designates 209.85.215.47 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAGnSx06gfkXxoaquU152BJAonUahsQ-rqJ37qV=-3-HxR4w_Zg@mail.gmail.com>
References: 
 <CAGQgZQTEnxuTPUgWOw=5Fhq3VuJ0Wv3qehyrM2q5FX1Qf+OCKg@mail.gmail.com>
	<CAGnSx06X9hBELt=qqHwqL0sXLwo6pU9L3GEiGPrOgYXBOpEoRg@mail.gmail.com>
	<CA+5TnhqUgS_CiyBWW5ugzvpkLVqOfmijKbyT93p6eD_zDHFctw@mail.gmail.com>
	<CAGQgZQT9FU6dgiitAfB3XGrM-pWAcopts5MkjL0EMOqtiLQGog@mail.gmail.com>
	<CAGnSx06gfkXxoaquU152BJAonUahsQ-rqJ37qV=-3-HxR4w_Zg@mail.gmail.com>
Date: Sat, 15 Dec 2012 17:20:02 -0500
Message-ID: 
 <CALis4o4-tNcKMrPuGLYNM9Fb-e5nNmTxdnNV05uaNJa2mqEd5A@mail.gmail.com>
Subject: Re: Create common module
From: Suraj Menon <menonsuraj5@gmail.com>
To: dev@hama.apache.org
Content-Type: multipart/alternative; boundary=bcaec54fb9d2aa2d0d04d0eb8c11

--bcaec54fb9d2aa2d0d04d0eb8c11
Content-Type: text/plain; charset=ISO-8859-1

+1 for hama-io, +0 for hama-commons. Can you look at the patch for HAMA-700
and let me know if you have any issues going forward in that direction.
Basically it includes a raw format to intermediate binary format
conversion, that could be overridden as shown in VertexInputReader
implementation. Now my question is, if the input to be read from HBase, do
we write every record from HBase to sequential file format? I am a little
confused here.

Thanks,
Suraj

On Fri, Dec 14, 2012 at 2:19 AM, Tommaso Teofili
<tommaso.teofili@gmail.com>wrote:

> thinking about it a bit further, maybe a separate module for IO related
> stuff may be worth, so hama-io instead of hama-commons.
> Whatever it is I'd like to avoid duplicated code.
> Tommaso
>
>
> 2012/12/13 Edward J. Yoon <edwardyoon@apache.org>
>
> > Otherwise, we have to copy VertexInputReader class to core module or
> > add own partitioner to graph package.
> >
> > In fact, I don't want to put any graph-related code to core module.
> > But if we want to support VertexInputReader, I think this is best.
> >
> > On Fri, Dec 14, 2012 at 4:22 AM, Apurv Verma <dapurv5@gmail.com> wrote:
> > > +0  I am not against but I think that would be an overkill for now.
> > >
> > > --
> > > Regards,
> > > Apurv Verma
> > >
> > >
> > >
> > >
> > > On Thu, Dec 13, 2012 at 12:32 PM, Tommaso Teofili
> > > <tommaso.teofili@gmail.com> wrote:
> > >> that sounds good to me, +1 for a hama-commons module.
> > >>
> > >> Tommaso
> > >>
> > >>
> > >> 2012/12/13 Edward J. Yoon <edwardyoon@apache.org>
> > >>
> > >>> Hi,
> > >>>
> > >>> I propose to create common module to share common classes among all
> > >>> modules.
> > >>>
> > >>> At the moment, the main reason is to support VertexInputReader. As I
> > >>> mentioned here[1], there's no way to integrate partitioner without
> > >>> merging graph to core or adding graph own partitioner. If we create
> > >>> common module, it'll all be clear. I'm not maven and java expert. If
> > >>> I'm wrong, please correct me.
> > >>>
> > >>> 1.
> > >>>
> >
> https://issues.apache.org/jira/browse/HAMA-531?focusedCommentId=13483009&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13483009
> > >>>
> > >>> --
> > >>> Best Regards, Edward J. Yoon
> > >>> @eddieyoon
> > >>>
> >
> >
> >
> > --
> > Best Regards, Edward J. Yoon
> > @eddieyoon
> >
>

--bcaec54fb9d2aa2d0d04d0eb8c11--