Mailing-List: contact hama-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: hama-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of tommaso.teofili@gmail.com
 designates 74.125.83.47 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAGQgZQRQncNKbrqVhuOChP_WF0ZsK+7tDe96Bt2XAgiQQ+usVA@mail.gmail.com>
References: 
 <CAGnSx05CRg1Qte1R15j+H0248NqiBN++kR=awWTUw40EQ3t0UQ@mail.gmail.com>
 <CAGnSx06Z8WOMJ-_BPjVBfFrfxQve=7resO_WYJ=mgz13w4MD+g@mail.gmail.com>
 <CAGQgZQQtYoaX-2FksVHPW2PvUaFHdevKoVy+E-zP-udpR5vm+g@mail.gmail.com>
 <CAGQgZQSnkabB0hOvRzS+MkNqAcKsAOLWC07umM+h2h8bN_a7fg@mail.gmail.com>
 <CAGnSx06yJ-g_OVoNLYUbWAv1u_Ms=9+-3Nw9B8M4GhEnqbSgSg@mail.gmail.com>
 <CAGQgZQQ0BqiFJw=E+CNT8o65AfE8i41j8akb2rOTQGy0CCJijQ@mail.gmail.com>
 <CAGQgZQR3HAeJsqXAg914jtkHFsHL3eKi3rMNq-X_NNsz0b7TmA@mail.gmail.com>
 <CAGnSx06e-PsT+_irMRMuQ+eggQM_b3=g_vN+fg1zzhtYHX-vTA@mail.gmail.com>
 <CAGnSx046cpwFAays_RDOaQ2WepEjbWbP3N_kJSoErgsPfAf3Fw@mail.gmail.com>
 <CAGQgZQRQncNKbrqVhuOChP_WF0ZsK+7tDe96Bt2XAgiQQ+usVA@mail.gmail.com>
From: Tommaso Teofili <tommaso.teofili@gmail.com>
Date: Mon, 1 Aug 2011 16:36:33 +0200
Message-ID: 
 <CAGnSx05=YXdkqL3rQWHBcZzFBRAY1dGmBDyAa9j=b_iVd5BYHA@mail.gmail.com>
Subject: Re: Refactoring Hama structure for better Maven usage
To: hama-dev@incubator.apache.org
Content-Type: multipart/alternative; boundary=000e0cd24cd821beba04a97291b6

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

I've committed the patch.
Everything seems fine to me.
Feedback appreciated :-)
Tommaso

p.s.
Maybe we should look at NOTICE/LICENSE files for each module by next release
to make sure everything is ok.

2011/8/1 Edward J. Yoon <edwardyoon@apache.org>

> +1
>
> On Mon, Aug 1, 2011 at 5:00 PM, Tommaso Teofili
> <tommaso.teofili@gmail.com> wrote:
> > I'll go on committing the patch by the end of the day if no one objects.
> > Tommaso
> >
> > 2011/8/1 Tommaso Teofili <tommaso.teofili@gmail.com>
> >
> >> Thanks Edward, I'll update the patch :)
> >> Tommaso
> >>
> >> 2011/8/1 Edward J. Yoon <edwardyoon@apache.org>
> >>
> >> >> I can't reproduce the error with testjob, Edward can you specify how
> to
> >>> >> reproduce that?
> >>>
> >>> I found it can be fixed by adding "System.getProperty("user.dir")" to
> >>> "TEST_JOB" path of TestBSPMasterGroomServer.
> >>>
> >>> 58 line: bsp.setJar(System.getProperty("user.dir") + "/" + TEST_JOB);
> >>>
> >>> On Fri, Jul 29, 2011 at 9:25 PM, Edward J. Yoon <edwardyoon@apache.org
> >
> >>> wrote:
> >>> > Yes sir~!
> >>> >
> >>> > On Fri, Jul 29, 2011 at 9:23 PM, Tommaso Teofili
> >>> > <tommaso.teofili@gmail.com> wrote:
> >>> >> I found myself a problem with src/webapps path written inside
> >>> HttpServer.
> >>> >> Will move src/webapps to src/main/webapp as this is the default
> Maven
> >>> >> location for jsp files.
> >>> >> I can't reproduce the error with testjob, Edward can you specify how
> to
> >>> >> reproduce that?
> >>> >> Thanks
> >>> >> Tommaso
> >>> >>
> >>> >> 2011/7/29 Edward J. Yoon <edwardyoon@apache.org>
> >>> >>
> >>> >>> Rest are very nice I'm +1 :D
> >>> >>>
> >>> >>> [INFO] Reactor Summary:
> >>> >>> [INFO]
> >>> >>> [INFO] Apache Hama parent POM ............................ SUCCESS
> >>> [0.691s]
> >>> >>> [INFO] Apache Hama Core .................................. SUCCESS
> >>> >>> [27.276s]
> >>> >>> [INFO] Apache Hama Examples .............................. SUCCESS
> >>> [4.733s]
> >>> >>> [INFO]
> >>> >>>
> >>>
> ------------------------------------------------------------------------
> >>> >>> [INFO] BUILD SUCCESS
> >>> >>> [INFO]
> >>> >>>
> >>>
> ------------------------------------------------------------------------
> >>> >>> [INFO] Total time: 32.837s
> >>> >>> [INFO] Finished at: Fri Jul 29 21:00:17 KST 2011
> >>> >>> [INFO] Final Memory: 21M/130M
> >>> >>>
> >>> >>> On Fri, Jul 29, 2011 at 9:03 PM, Edward J. Yoon <
> >>> edwardyoon@apache.org>
> >>> >>> wrote:
> >>> >>> > I found one error message: "File
> src/test/java/testjar/testjob.jar
> >>> >>> > does not exist."
> >>> >>> >
> >>> >>> > Does anyone have a idea to get rid of this jar file? or to get
> base
> >>> >>> directory?
> >>> >>> >
> >>> >>> > On Fri, Jul 29, 2011 at 7:44 PM, Tommaso Teofili
> >>> >>> > <tommaso.teofili@gmail.com> wrote:
> >>> >>> >> Hello again,
> >>> >>> >> I've opened HAMA-417 [1] and supplied a patch.
> >>> >>> >> Before committing it, I ask you to review it as it's a pretty
> huge
> >>> >>> >> refactoring.
> >>> >>> >> Then I can go on either committing it on trunk or creating a
> >>> separate
> >>> >>> branch
> >>> >>> >> which can be merged back to trunk afterwards.
> >>> >>> >> Cheers,
> >>> >>> >> Tommaso
> >>> >>> >>
> >>> >>> >> [1] : https://issues.apache.org/jira/browse/HAMA-417
> >>> >>> >>
> >>> >>> >> 2011/7/29 Tommaso Teofili <tommaso.teofili@gmail.com>
> >>> >>> >>
> >>> >>> >>> Hi al,
> >>> >>> >>> I'd like to start the refactoring we discussed some time ago in
> >>> order
> >>> >>> to
> >>> >>> >>> create different modules for hama-core and hama-examples, this
> >>> should
> >>> >>> also
> >>> >>> >>> help any future factoring out of functionalities in more fine
> >>> grained
> >>> >>> >>> modules.
> >>> >>> >>> What do you think?
> >>> >>> >>> Regards,
> >>> >>> >>> Tommaso
> >>> >>> >>>
> >>> >>> >>
> >>> >>> >
> >>> >>> >
> >>> >>> >
> >>> >>> > --
> >>> >>> > Best Regards, Edward J. Yoon
> >>> >>> > @eddieyoon
> >>> >>> >
> >>> >>>
> >>> >>>
> >>> >>>
> >>> >>> --
> >>> >>> Best Regards, Edward J. Yoon
> >>> >>> @eddieyoon
> >>> >>>
> >>> >>
> >>> >
> >>> >
> >>> >
> >>> > --
> >>> > Best Regards, Edward J. Yoon
> >>> > @eddieyoon
> >>> >
> >>>
> >>>
> >>>
> >>> --
> >>> Best Regards, Edward J. Yoon
> >>> @eddieyoon
> >>>
> >>
> >>
> >
>
>
>
> --
> Best Regards, Edward J. Yoon
> @eddieyoon
>

--000e0cd24cd821beba04a97291b6--