Hi Niranda, This is merged now. @Maheshika, can you please deploy these orbit bundles? There are five of them I believe.
Thanks. On Tue, Feb 24, 2015 at 5:36 PM, Niranda Perera <nira...@wso2.com> wrote: > Hi Kasun, > > May I know the progress of this? > > Cheers > > On Tue, Feb 24, 2015 at 4:26 PM, Niranda Perera <nira...@wso2.com> wrote: > >> Hi Kasun, >> >> Sorry my mistake. I put those in to test an orbit build locally. forgot >> to remove it in the commit. >> >> https://github.com/wso2/orbit/pull/65 >> >> On Tue, Feb 24, 2015 at 4:20 PM, KasunG Gajasinghe <kas...@wso2.com> >> wrote: >> >>> Hi, >>> >>> Did you push all the changes? I still see that root pom has sub-module >>> references. >>> >>> >>> On Tue, Feb 24, 2015 at 4:05 PM, Niranda Perera <nira...@wso2.com> >>> wrote: >>> >>>> Hi Kasun, >>>> >>>> here are the changes I did. >>>> >>>> 1. done >>>> 2. changed >>>> org.wso2.apache.spark >>>> org.wso2.apache.hadoop >>>> org.wso2.clearspring.analytics >>>> org.wso2.json4s >>>> org.wso2.uncommons.maths >>>> org.wso2.twitter >>>> >>>> 3. done >>>> >>>> 4. the todo was to resolve the private packages warnings. But I have to >>>> include the private packages. so, I removed the todo item >>>> >>>> 5. that was how it was done in the 1.1.0 orbit bundle. but the jar >>>> imports are optional. they are already in the spark core resources, so >>>> deleted those. >>>> >>>> 6. uncommon-maths do not need any imports >>>> >>>> Could you please check it. [1] >>>> >>>> [1] https://github.com/wso2/orbit/pull/65 >>>> >>>> Thanks >>>> >>>> On Tue, Feb 24, 2015 at 1:10 PM, KasunG Gajasinghe <kas...@wso2.com> >>>> wrote: >>>> >>>>> Hi Niranda, >>>>> >>>>> Please do the following sugestions on these orbit bundles. There's a >>>>> quite number of orbits here, hence the long list! >>>>> >>>>> 1. Set the dependencies as optional. >>>>> >>>>> 2. The groupId should start with org.wso2.*. This is a recent addition >>>>> to policy to make our orbit bundles compatible with maven central artifact >>>>> policies. >>>>> >>>>> 3. Remove the references from parent orbit pom >>>>> >>>>> 4. There's a TODO item in the spark-core_2.10 orbit. Can you fix that >>>>> before releasing this? >>>>> >>>>> 5. Why are the conf files created in spark-core_2.10 orbit. It looks >>>>> like this is coming from a third-party jar. Please use <Include-Resource> >>>>> instruction for this as well like you have done for other config files. >>>>> >>>>> 6. Use explicit imports in uncommons-maths orbit. >>>>> >>>>> Thanks. >>>>> >>>>> >>>>> On Tue, Feb 24, 2015 at 12:45 PM, KasunG Gajasinghe <kas...@wso2.com> >>>>> wrote: >>>>> >>>>>> >>>>>> Sure Sameera. >>>>>> >>>>>> On Tue, Feb 24, 2015 at 12:32 PM, Sameera Jayasoma <same...@wso2.com> >>>>>> wrote: >>>>>> >>>>>>> KasunG can you have a look at this. >>>>>>> >>>>>>> >>>>>>> On Tue, Feb 24, 2015 at 10:20 AM, Niranda Perera <nira...@wso2.com> >>>>>>> wrote: >>>>>>> >>>>>>>> Hi Sameera, >>>>>>>> >>>>>>>> Please review and merge this PR [1] >>>>>>>> >>>>>>>> It adds the spark core and sql v1.2.1 to the orbit together with, >>>>>>>> hadoop-client v2.2.0 >>>>>>>> Uncommons Math v1.2.2 >>>>>>>> json4s-jackson_2.10 v3.2.10 >>>>>>>> twitter chill and java v0.5.0 >>>>>>>> stream v2.2.0 >>>>>>>> >>>>>>>> Would be very grateful if you could attend to this ASAP because we >>>>>>>> have scheduled a M1 release by the EOD today 24th Feb. >>>>>>>> >>>>>>>> I have reviewed the spark core and sql bundles with KIshanthan last >>>>>>>> week. Much appreciate if you could go through the poms nevertheless. >>>>>>>> >>>>>>>> cheers >>>>>>>> >>>>>>>> [1] https://github.com/wso2/orbit/pull/65 >>>>>>>> >>>>>>>> -- >>>>>>>> *Niranda Perera* >>>>>>>> Software Engineer, WSO2 Inc. >>>>>>>> Mobile: +94-71-554-8430 >>>>>>>> Twitter: @n1r44 <https://twitter.com/N1R44> >>>>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> Sameera Jayasoma, >>>>>>> Software Architect, >>>>>>> >>>>>>> WSO2, Inc. (http://wso2.com) >>>>>>> email: same...@wso2.com >>>>>>> blog: http://blog.sameera.org >>>>>>> twitter: https://twitter.com/sameerajayasoma >>>>>>> flickr: http://www.flickr.com/photos/sameera-jayasoma/collections >>>>>>> Mobile: 0094776364456 >>>>>>> >>>>>>> Lean . Enterprise . Middleware >>>>>>> >>>>>>> >>>>>> >>>>>> >>>>>> -- >>>>>> >>>>>> *Kasun Gajasinghe*Senior Software Engineer, WSO2 Inc. >>>>>> email: kasung AT spamfree wso2.com >>>>>> linked-in: http://lk.linkedin.com/in/gajasinghe >>>>>> blog: http://kasunbg.org >>>>>> >>>>>> >>>>>> >>>>> >>>>> >>>>> >>>>> -- >>>>> >>>>> *Kasun Gajasinghe*Senior Software Engineer, WSO2 Inc. >>>>> email: kasung AT spamfree wso2.com >>>>> linked-in: http://lk.linkedin.com/in/gajasinghe >>>>> blog: http://kasunbg.org >>>>> >>>>> >>>>> >>>> >>>> >>>> >>>> -- >>>> *Niranda Perera* >>>> Software Engineer, WSO2 Inc. >>>> Mobile: +94-71-554-8430 >>>> Twitter: @n1r44 <https://twitter.com/N1R44> >>>> >>> >>> >>> >>> -- >>> >>> *Kasun Gajasinghe*Senior Software Engineer, WSO2 Inc. >>> email: kasung AT spamfree wso2.com >>> linked-in: http://lk.linkedin.com/in/gajasinghe >>> blog: http://kasunbg.org >>> >>> >>> >> >> >> >> -- >> *Niranda Perera* >> Software Engineer, WSO2 Inc. >> Mobile: +94-71-554-8430 >> Twitter: @n1r44 <https://twitter.com/N1R44> >> > > > > -- > *Niranda Perera* > Software Engineer, WSO2 Inc. > Mobile: +94-71-554-8430 > Twitter: @n1r44 <https://twitter.com/N1R44> > -- *Kasun Gajasinghe*Senior Software Engineer, WSO2 Inc. email: kasung AT spamfree wso2.com linked-in: http://lk.linkedin.com/in/gajasinghe blog: http://kasunbg.org
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev