Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <CAHG4OXsYvagkBpFK5R_RyXoyNur76HveTO73fo0=cpnYTVWz-g@mail.gmail.com>
References: 
 <CAKoNANtx7OBboOobM-L=vftbrWSJsyg8XXPRn881kB_QDb9x1g@mail.gmail.com>
	<CAKoNANu8stmWm-yeMm0dNk5vdhduK0asBD0D-JHAS4enOmUe_Q@mail.gmail.com>
	<CA+0=VoVmtioyqu3EcdN+pJTwvUBJcMoM1CA2tfSj935Ngax2sA@mail.gmail.com>
	<CAKoNANtedwwzWH5WkRThdnp-Q3hgyC35Wh7xSNnUXd1xW23umA@mail.gmail.com>
	<CAGcMBHgazccx6rtuAy79wFNPM7pzUKpoatXLZuRipmq-bsJSGg@mail.gmail.com>
	<CAOKDqC_z4dZPS6DeWSV+zHk07mSNvKSkuWG1rHwc6Agu8kZneg@mail.gmail.com>
	<CA+0=VoWh1YS0mrPVysyDu11hs4O4HGS=gerQEuLWO1g1=m4-gw@mail.gmail.com>
	<CAHG4OXsYvagkBpFK5R_RyXoyNur76HveTO73fo0=cpnYTVWz-g@mail.gmail.com>
Date: Thu, 3 Dec 2015 14:12:30 +0300
Message-ID: 
 <CAGcMBHhCYT7dGp3zy0GbfifcVnREJ8s9bxUx634WW+U5=mh6RQ@mail.gmail.com>
Subject: Re: OSGi integration ready for review
From: Yakov Zhdanov <yzhdanov@apache.org>
To: dev@ignite.apache.org
Content-Type: multipart/alternative; boundary=001a114009c68a94a60525fc77bc

--001a114009c68a94a60525fc77bc
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Raul, I did review as well. I added my comments to the ticket and committed
some changes to ignite-1270.

Given that this feature is desperately desired, can we merge it to 1.5? To
do that you can squash all the differences between 1270 and master and pick
that commit to ignite-1.5. If someone knows better way to do this, please
let us know.

I think 1.5-final can be released in 2 weeks, i.e. around Dec, 18.

--Yakov

2015-12-03 12:20 GMT+03:00 Romain Gilles <romain.gilles@gmail.com>:

> Hi Igniters,
> I did the review. I term of impact the Raul commits change the manifest o=
f
> the overall produced jar files to introduce OSGi specific headers. This
> will not impact original ignite usage as OSGi headers does are only for
> OSGi environments. Some times maven-bundle-plugin can have side effect on
> split package but I hope Raul has already checked it.
> Then in term of code Raul has only added new project and code. He didn't
> modify a single line of code in the already existing project therefore it
> should not impact the existing code has it didn't touch it.
> The only things I mention to Raul is to double check that the user class
> loader (here Raul provide an OSGi aware user classloader) is used every
> where in the serialization process:
>  - Binary marshaller (new)
>  - Optimized marshaller (old)
> But here again it is just to make sure that in OSGi environment the new
> feature provided by Raul will work without bad surprise it doesn't impact
> the existing code.
>
>
> Other than that it is a great feature and I'm excited to be able to use i=
t!
>
> Romain.
>
> PS: some enhancement will be added in the future as discussed in the jira
> ticket during the review. But it is an iterative approach and for now wha=
t
> is provide is enough to start!
>
> Le jeu. 3 d=C3=A9c. 2015 =C3=A0 02:43, Dmitriy Setrakyan <dsetrakyan@apac=
he.org> a
> =C3=A9crit :
>
> > Guys,
> >
> > Should we try to review and merge the OSGI integration before the final
> > release? it does sound like it is additive and should not affect other
> > functionality.
> >
> > My preference would be to get it in, so other OSGI-related integrations
> > (e.g. Camel) would not have to wait for 1.6.
> >
> > Thoughts?
> > D.
> >
> > On Mon, Nov 30, 2015 at 11:48 PM, Anton Vinogradov <
> > avinogradov@gridgain.com
> > > wrote:
> >
> > > Raul,
> > >
> > > I've added 1.6 space as a copy of 1.5.
> > > But I gain no results at OSGi search.
> > > Could you please specify what pages should be removed from 1.5 or
> revome
> > > them by yourself?
> > >
> > > On Mon, Nov 30, 2015 at 6:18 PM, Yakov Zhdanov <yzhdanov@apache.org>
> > > wrote:
> > >
> > > > Raul, as far as commits - they are mostly fixes for functionality,
> but
> > do
> > > > not add new features.
> > > >
> > > > Can someone create 1.6 space and move OSGi articles there? Perhaps,
> > Anton
> > > > V. can do it?
> > > >
> > > > --Yakov
> > > >
> > > > 2015-11-30 16:42 GMT+03:00 Raul Kripalani <raulk@apache.org>:
> > > >
> > > > > Dmitriy,
> > > > >
> > > > > I'm not going to judge if the communication of release timelines =
is
> > > > > efficient. In fact, the release has still not been made and there=
's
> > no
> > > > > traffic in the ML about what's going on. Commits are still being
> > > pushed,
> > > > no
> > > > > one knows anything. It seems communication is internal between
> > > > committers.
> > > > > But anyway, let's move ahead with OSGi.
> > > > >
> > > > > I have merged 1.5 into master, and master into ignite-1270. So it=
's
> > now
> > > > > mergeable into 1.6
> > > > >
> > > > > Someone needs to remove OSGi from the readme.io and push it into
> 1.6
> > > > (not
> > > > > me, I'm not happy with all this).
> > > > >
> > > > > Regards,
> > > > >
> > > > > *Ra=C3=BAl Kripalani*
> > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big
> Data
> > > and
> > > > > Messaging Engineer
> > > > > http://about.me/raulkripalani |
> > > http://www.linkedin.com/in/raulkripalani
> > > > > http://blog.raulkr.net | twitter: @raulvk
> > > > >
> > > > > On Sat, Nov 28, 2015 at 5:13 AM, Dmitriy Setrakyan <
> > > > dsetrakyan@apache.org>
> > > > > wrote:
> > > > >
> > > > > > On Fri, Nov 27, 2015 at 5:48 PM, Raul Kripalani <
> raulk@apache.org>
> > > > > wrote:
> > > > > >
> > > > > > > Aside from Dmitriy's questions, was there any blocker for
> someone
> > > to
> > > > > > review
> > > > > > > the code? There has been no activity from committers.
> > > > > > >
> > > > > >
> > > > > > As I mentioned last week, we were planning to cut a release thi=
s
> > > > Friday,
> > > > > > but I don=E2=80=99t think it happened. I too would like to get =
an update.
> > > > Yakov,
> > > > > > can you send out the status on open issues?
> > > > > >
> > > > > >
> > > > > > > It was supposed to be merged for 1.5, and as far as I know,
> there
> > > was
> > > > > no
> > > > > > > official announcement in the dev@ list that the release was
> > being
> > > > cut
> > > > > > > today. How is the dev team supposed to be synchronised around
> > these
> > > > > > > milestones???
> > > > > > >
> > > > > >
> > > > > > Raul, I am hoping the release will happen over the weekend,
> > assuming
> > > we
> > > > > get
> > > > > > an update. Even in that case, forcefully merging a new feature,
> > that
> > > is
> > > > > not
> > > > > > a critical bug fix, at this point would be too risky.
> > > > > >
> > > > > > My preference would be to cut another release with OSGI, Cammel=
,
> > and
> > > > > > Node.JS early January (no point of doing it sooner due to
> > holidays).
> > > > > Would
> > > > > > do you think?
> > > > > >
> > > > > >
> > > > > > >
> > > > > > > To the release manager: please take out the OSGi documentatio=
n
> > from
> > > > 1.5
> > > > > > in
> > > > > > > readme.io and put it in 1.6.
> > > > > > >
> > > > > > > Regards,
> > > > > > >
> > > > > > > *Ra=C3=BAl Kripalani*
> > > > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration,
> Big
> > > Data
> > > > > and
> > > > > > > Messaging Engineer
> > > > > > > http://about.me/raulkripalani |
> > > > > http://www.linkedin.com/in/raulkripalani
> > > > > > > http://blog.raulkr.net | twitter: @raulvk
> > > > > > >
> > > > > > > On Tue, Nov 24, 2015 at 8:11 PM, Raul Kripalani <
> > raulk@apache.org>
> > > > > > wrote:
> > > > > > >
> > > > > > > > Hello all,
> > > > > > > >
> > > > > > > > The initial OSGi integration is ready for review in branch
> > > > > ignite-1270.
> > > > > > > > Features included:
> > > > > > > >
> > > > > > > > * OSGi manifest for most modules. I could OSGi-fy 22 module=
s.
> > > Some
> > > > > are
> > > > > > > > left out due to complexity or not supporting OSGi: hadoop,
> > mesos,
> > > > > > cloud,
> > > > > > > > gce, yarn, etc.
> > > > > > > > * Karaf features repository with 1 feature per module + a
> > global
> > > > > > > > ignite-all feature. Each feature installs the required
> > > dependencies
> > > > > > into
> > > > > > > > the container.
> > > > > > > > * Ignite OSGi Bundle Activator which activates an Ignite OS=
Gi
> > > > > > > > configuration and injects the appropriate OSGi-friendly
> > > > classloader.
> > > > > > > > * Two classloading strategies for OSGi:
> > > > > > > >     * Bundle Delegating =3D> attempts to find the class in =
the
> > > bundle
> > > > > > that
> > > > > > > > loaded Ignite, falling back to ignite-core.
> > > > > > > >     * Container Sweep =3D> same as above, but sweeps the en=
tire
> > > OSGi
> > > > > > > > container for the first hit as the ultimate fallback (keeps=
 a
> > > > cache).
> > > > > > > >
> > > > > > > > (@Romain =E2=80=93 the latter should work for you).
> > > > > > > >
> > > > > > > > Since generating the Karaf features repo (features.xml)
> > requires
> > > > > > > including
> > > > > > > > version numbers of the 3rd party deps, I have promoted the
> > > version
> > > > > > > numbers
> > > > > > > > I needed to parent/pom.xml as Maven properties. This is als=
o
> a
> > > > Maven
> > > > > > best
> > > > > > > > practice.
> > > > > > > >
> > > > > > > > In this branch, IGNITE-1810 is also included. I was able to
> > test
> > > it
> > > > > > > > locally only, given that the CI test job is broken in
> TeamCity.
> > > > > > > >
> > > > > > > > Note: Spring configuration of Ignite is not supported due t=
o
> > > Spring
> > > > > DM
> > > > > > > > 1.2.1 not being compatible with Spring Framework 4.x anymor=
e.
> > > This
> > > > is
> > > > > > > being
> > > > > > > > discussed at the SMX community [1]. For now, we only suppor=
t
> > > > > > programmatic
> > > > > > > > configuration.
> > > > > > > >
> > > > > > > > As you know, I created a user guide section for OSGi in the
> 1.5
> > > > docs
> > > > > > [2].
> > > > > > > >
> > > > > > > > I'd like this feature to target 1.5. It does modify existin=
g
> > > Ignite
> > > > > > > > codebase, so it should be pretty safe to release.
> > > > > > > >
> > > > > > > > [1]
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> http://servicemix.396122.n5.nabble.com/Spring-DM-for-Spring-4-in-SMX-Kara=
f-td5723130.html
> > > > > > > > [2] https://dash.readme.io/project/apacheignite/v1.5
> > > > > > > >
> > > > > > > > *Ra=C3=BAl Kripalani*
> > > > > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration=
,
> > Big
> > > > Data
> > > > > > and
> > > > > > > > Messaging Engineer
> > > > > > > > http://about.me/raulkripalani |
> > > > > > http://www.linkedin.com/in/raulkripalani
> > > > > > > > http://blog.raulkr.net | twitter: @raulvk
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>

--001a114009c68a94a60525fc77bc--