Return-Path: X-Original-To: apmail-airavata-dev-archive@www.apache.org Delivered-To: apmail-airavata-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 8F8AF10FEB for ; Tue, 4 Feb 2014 16:14:30 +0000 (UTC) Received: (qmail 70090 invoked by uid 500); 4 Feb 2014 16:14:29 -0000 Delivered-To: apmail-airavata-dev-archive@airavata.apache.org Received: (qmail 69971 invoked by uid 500); 4 Feb 2014 16:14:28 -0000 Mailing-List: contact dev-help@airavata.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@airavata.apache.org Delivered-To: mailing list dev@airavata.apache.org Received: (qmail 69963 invoked by uid 99); 4 Feb 2014 16:14:28 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 04 Feb 2014 16:14:28 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of glahiru@gmail.com designates 209.85.160.51 as permitted sender) Received: from [209.85.160.51] (HELO mail-pb0-f51.google.com) (209.85.160.51) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 04 Feb 2014 16:14:24 +0000 Received: by mail-pb0-f51.google.com with SMTP id un15so8687417pbc.10 for ; Tue, 04 Feb 2014 08:14:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=Vq7u6khOsL5yUysx+p3riJI8Se21J/PJTYPECIZSPHI=; b=Gy09GrKHzwsWiG8A3Z9mt/k8VzHrTRMeI1d9QcSSmieY3ZLo6b8N70LziY8DVJTUCH MWL4ySRsf1ZJeSzyPZFVqJ2EvlsYKuAsDBLL1gtPK90tvG4zxda9D+e43OWuBi/cOP8t Xly5XZ1c8BU7e2cdxOz2TtlHr6MI0+z1RmVQdTfzxI14lB7F8wN0olSkSC0o2XWTyyc0 vQx+x1yvwoaqPfZBAwwh+QrmmoodnxqN3cMp7GU3QqvoLlPuDsf2mxPMJVuNeev0vujO eoJsPlQTzHqxlHt2rES7yWsuOgvPNJfj+2f0/L+RcB+UtmLYjEoFniPpWbHYfcqolRaZ atVA== MIME-Version: 1.0 X-Received: by 10.66.142.233 with SMTP id rz9mr45007398pab.71.1391530438697; Tue, 04 Feb 2014 08:13:58 -0800 (PST) Received: by 10.66.14.193 with HTTP; Tue, 4 Feb 2014 08:13:58 -0800 (PST) In-Reply-To: References: <52EFA43A.3080909@iu.edu> <52EFDA38.6060002@iu.edu> <24D7F226-D5A0-48DC-8FD0-6EC2D3FA7EE5@apache.org> <52F0FAC1.1040903@iu.edu> Date: Tue, 4 Feb 2014 11:13:58 -0500 Message-ID: Subject: Re: Hangout: Orchestrator, Registry, GFAC, and API recap and plans From: Lahiru Gunathilake To: dev Content-Type: multipart/alternative; boundary=001a113448087e29ee04f196ed16 X-Virus-Checked: Checked by ClamAV on apache.org --001a113448087e29ee04f196ed16 Content-Type: text/plain; charset=ISO-8859-1 Hi Suresh, Does thrift has a maven plugin to generate classes ? We can have interfaces in one module and different implementations in a separate module. If there's a change (API level change, the implementation modules should be changed, otherwise they shouldn't be compiled). But if someone wants the old one they can have a dependency to the old module and should be able to compile the impl module). Suresh, WDYT ? Regards Lahiru On Tue, Feb 4, 2014 at 11:10 AM, Suresh Marru wrote: > On Feb 4, 2014, at 9:59 AM, Lahiru Gunathilake wrote: > > > Hi Suresh, > > > > Current airavataAPI looks very clean and easy to understand. Thanks to > your very detail comments I was able to understand the process. > > > > I think we need to make sure we give a simple easy to use > experimentModel.ConfigurationData object to configure. What is our plan > with implementing experimentModel.ConfigurationData ? > > Yes, we need every one's critical eye in making these configuration data > to cover all the use cases, yet be simple and as concise as possible. > > For the implementation, thrift has a generator for java beans (in addition > to standard java) which essentially makes all members to private with > setters to return void. We can have this generated beans as a separate > airavata-thrift-datamodel.jar and all components will use this the same way > we now use workflow execution context or workflow tracking library. Not > sure if this is what you asked. > > > Regards > > Lahiru > > > > > > On Tue, Feb 4, 2014 at 9:35 AM, Marlon Pierce wrote: > > Where do these go? In modules/thrift-interfaces? > > I am now poking around at other thrift based projects to find a way to > organize these. A challenge I see if how to over-write the generated > classes without loosing any implementations. Will post a layout for > discussion soon. > > Suresh > > > > > > Marlon > > > > On 2/4/14 9:34 AM, Suresh Marru wrote: > > > Makes sense Lahiru. I will commit generated data model into commons > and client stubs and server skeletons. > > > > > > Suresh > > > > > > On Feb 4, 2014, at 9:18 AM, Lahiru Gunathilake > wrote: > > > > > >> Hi Suresh, > > >> > > >> I think to do a proper review, we need to generate code and commit to > the repository. This will make life easier for developers, because > everybody doesn't have to go through the hard steps to install Thrift in > their machines. > > >> > > >> By looking in to .thrift files its difficult to navigate through the > object model and hard to understand it. > > >> > > >> Regards > > >> Lahiru > > >> > > >> > > >> On Mon, Feb 3, 2014 at 1:04 PM, Marlon Pierce > wrote: > > >> Hangout URL is > > >> https://plus.google.com/hangouts/_/72cpjvt6sl94fnn4hs10bibi6g?hl=en > > >> > > >> > > >> Marlon > > >> > > >> On 2/3/14 9:14 AM, Marlon Pierce wrote: > > >>> Hi all-- > > >>> > > >>> We'll have a hangout at 1:00 pm US EST to discuss the subject. > Hangout > > >>> URL will be forthcoming. > > >>> > > >>> > > >>> Marlon > > >>> > > >> > > >> > > >> > > >> -- > > >> System Analyst Programmer > > >> PTI Lab > > >> Indiana University > > > > > > > > > > -- > > System Analyst Programmer > > PTI Lab > > Indiana University > > -- System Analyst Programmer PTI Lab Indiana University --001a113448087e29ee04f196ed16 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable
Hi Suresh,

Does thrift has a maven plug= in to generate classes ? We can have interfaces in one module and different= implementations in a separate module. If there's a change (API level c= hange, the implementation modules should be changed, otherwise they shouldn= 't be compiled). But if someone wants the old one they can have a depen= dency to the old module and should be able to compile the impl module).

Suresh, WDYT ?

Regards
Lahiru


On Tue, Feb 4, 2014 at 11:10 AM, Suresh Marru <= ;smarru@apache.org> wrote:
On Feb 4, 2014, at 9:59 AM= , Lahiru Gunathilake <glahiru@gmail= .com> wrote:

> Hi Suresh,
>
> Current airavataAPI looks very clean and easy = to understand. Thanks to your very detail comments I was able to understand= the process.
>
> I think we need to make sure we give a simple easy to use experimentMo= del.ConfigurationData object to configure. What is our plan with implementi= ng experimentModel.ConfigurationData ?

Yes, we need every one’s critical eye in making these configura= tion data to cover all the use cases, yet be simple and as concise as possi= ble.

For the implementation, thrift has a generator for java beans (in addition = to standard java) which essentially makes all members to private with sette= rs to return void. We can have this generated beans as a separate airavata-= thrift-datamodel.jar and all components will use this the same way we now u= se workflow execution context or workflow tracking library. Not sure if thi= s is what you asked.

> Regards
> Lahiru
>
>
> On Tue, Feb 4, 2014 at 9:35 AM, Marlon Pierce <marpierc@iu.edu> wrote:
> Where do these go?  In modules/thrift-interfaces?

I am now poking around at other thrift based projects to find a way t= o organize these. A challenge I see if how to over-write the generated clas= ses without loosing any implementations. Will post a layout for discussion = soon.

Suresh


>
> Marlon
>
> On 2/4/14 9:34 AM, Suresh Marru wrote:
> > Makes sense Lahiru. I will commit generated data model into commo= ns and client stubs and server skeletons.
> >
> > Suresh
> >
> > On Feb 4, 2014, at 9:18 AM, Lahiru Gunathilake <glahiru@gmail.com> wrote:
> >
> >> Hi Suresh,
> >>
> >> I think to do a proper review, we need to generate code and c= ommit to the repository. This will make life easier for developers, because= everybody doesn't have to go through the hard steps to install Thrift = in their machines.
> >>
> >> By looking in to .thrift files its difficult to navigate thro= ugh the object model and hard to understand it.
> >>
> >> Regards
> >> Lahiru
> >>
> >>
> >> On Mon, Feb 3, 2014 at 1:04 PM, Marlon Pierce <marpierc@iu.edu> wrote:
> >> Hangout URL is
> >> https://plus.google.com/hangouts/_/7= 2cpjvt6sl94fnn4hs10bibi6g?hl=3Den
> >>
> >>
> >> Marlon
> >>
> >> On 2/3/14 9:14 AM, Marlon Pierce wrote:
> >>> Hi all--
> >>>
> >>> We'll have a hangout at 1:00 pm US EST to discuss the= subject.  Hangout
> >>> URL will be forthcoming.
> >>>
> >>>
> >>> Marlon
> >>>
> >>
> >>
> >>
> >> --
> >> System Analyst Programmer
> >> PTI Lab
> >> Indiana University
>
>
>
>
> --
> System Analyst Programmer
> PTI Lab
> Indiana University




--
= System Analyst Programmer
PTI Lab
Indiana University
--001a113448087e29ee04f196ed16--