Thanks artem. Sorry, was not aware of the commit schedule policy On 19 Jun 2015 23:39, "Artiom Shutak" <ashu...@gridgain.com> wrote:
> Yes, I've completed changes that I wanted to do. > > Someone from commiters will review all changes and it will be merged at > sprint-7 (approximately in Monday). > > I let you know. > > -- Artem -- > > On Fri, Jun 19, 2015 at 6:40 PM, Atri Sharma <atri.j...@gmail.com> wrote: > > > Folks, > > > > Any update on this please? > > On 11 Jun 2015 15:14, "Atri Sharma" <atri.j...@gmail.com> wrote: > > > > > That makes sense, thanks! > > > > > > Thanks for your review! > > > > > > On Thu, Jun 11, 2015 at 3:13 PM, Artiom Shutak <ashu...@gridgain.com> > > > wrote: > > > > > >> The main part of needed fixes is about the test for new functionality. > > It > > >> will have some hacks and tricks, so I think It would be more simpler > if > > >> you > > >> will look on it after it will be fixed. > > >> > > >> -- Artem -- > > >> > > >> On Thu, Jun 11, 2015 at 12:33 PM, Atri Sharma <atri.j...@gmail.com> > > >> wrote: > > >> > > >> > Thanks! > > >> > > > >> > What are the fixes needed as yet? I can do them now... > > >> > > > >> > On Thu, Jun 11, 2015 at 3:01 PM, Artiom Shutak < > ashu...@gridgain.com> > > >> > wrote: > > >> > > > >> > > Atri, > > >> > > > > >> > > I am ok with your patch in general. > > >> > > > > >> > > I think, someone from commiters will create new branch with your > > >> patch, > > >> > do > > >> > > all needed fixes and merge it in a couple of days. > > >> > > > > >> > > Thank you for your conrtibution at Ignite! > > >> > > > > >> > > -- Artem -- > > >> > > > > >> > > On Wed, Jun 10, 2015 at 4:45 PM, Atri Sharma <atri.j...@gmail.com > > > > >> > wrote: > > >> > > > > >> > > > Please let me know your comments and feedback. > > >> > > > On 9 Jun 2015 23:38, "Artiom Shutak" <ashu...@gridgain.com> > > wrote: > > >> > > > > > >> > > > > Atri, see my comments at jira. > > >> > > > > > > >> > > > > I think someone from commiters should do review. > > >> > > > > > > >> > > > > -- Artem -- > > >> > > > > > > >> > > > > On Tue, Jun 9, 2015 at 7:56 PM, Atri Sharma < > > atri.j...@gmail.com> > > >> > > wrote: > > >> > > > > > > >> > > > > > Updated patch sent. > > >> > > > > > > > >> > > > > > Please see and let me know your feedback and comments. > > >> > > > > > > > >> > > > > > On Tue, Jun 9, 2015 at 12:32 AM, Artiom Shutak < > > >> > ashu...@gridgain.com > > >> > > > > > >> > > > > > wrote: > > >> > > > > > > > >> > > > > > > See comments at Jira. > > >> > > > > > > > > >> > > > > > > -- Artem -- > > >> > > > > > > > > >> > > > > > > On Mon, Jun 8, 2015 at 8:21 PM, Atri Sharma < > > >> atri.j...@gmail.com > > >> > > > > >> > > > > wrote: > > >> > > > > > > > > >> > > > > > > > Latest patch uploaded. Please see and let me know your > > >> > comments > > >> > > > > > > > On 8 Jun 2015 22:19, "Atri Sharma" <atri.j...@gmail.com > > > > >> > wrote: > > >> > > > > > > > > > >> > > > > > > > > Thanks for that. > > >> > > > > > > > > > > >> > > > > > > > > So got a build success. > > >> > > > > > > > > > > >> > > > > > > > > On Mon, Jun 8, 2015 at 10:07 PM, Artiom Shutak < > > >> > > > > ashu...@gridgain.com > > >> > > > > > > > > >> > > > > > > > > wrote: > > >> > > > > > > > > > > >> > > > > > > > >> Atri, > > >> > > > > > > > >> > > >> > > > > > > > >> As usual, then you are using Apache Maven then BUILD > > >> SUCCESS > > >> > > > means > > >> > > > > > all > > >> > > > > > > > are > > >> > > > > > > > >> OK, and BUILD FAILURE means that something went > wrong. > > >> > > > > > > > >> > > >> > > > > > > > >> -- Artem -- > > >> > > > > > > > >> > > >> > > > > > > > >> On Mon, Jun 8, 2015 at 7:20 PM, Atri Sharma < > > >> > > > atri.j...@gmail.com> > > >> > > > > > > > wrote: > > >> > > > > > > > >> > > >> > > > > > > > >> > Thanks. > > >> > > > > > > > >> > > > >> > > > > > > > >> > I have fixed the comments. > > >> > > > > > > > >> > > > >> > > > > > > > >> > My local results for your command are: > > >> > > > > > > > >> > > > >> > > > > > > > >> > [21:25:16,252][INFO > > >> ][Thread-2][GridProjectionSelfTest0] > > >> > > > > > > > >> > > > >> > > > > > > > >> > >>> > > >> > +-----------------------------------------------------+ > > >> > > > > > > > >> > >>> Ignite ver. 1.1.1-SNAPSHOT#19700101-sha1:DEV > > >> stopped > > >> > OK > > >> > > > > > > > >> > >>> > > >> > +-----------------------------------------------------+ > > >> > > > > > > > >> > >>> Grid name: internal.GridProjectionSelfTest0 > > >> > > > > > > > >> > >>> Grid uptime: 00:03:15:461 > > >> > > > > > > > >> > > > >> > > > > > > > >> > > > >> > > > > > > > >> > [21:25:16,252][INFO > > >> ][Thread-8][GridProjectionSelfTest3] > > >> > > > > > > > >> > > > >> > > > > > > > >> > >>> > > >> > +-----------------------------------------------------+ > > >> > > > > > > > >> > >>> Ignite ver. 1.1.1-SNAPSHOT#19700101-sha1:DEV > > >> stopped > > >> > OK > > >> > > > > > > > >> > >>> > > >> > +-----------------------------------------------------+ > > >> > > > > > > > >> > >>> Grid name: internal.GridProjectionSelfTest3 > > >> > > > > > > > >> > >>> Grid uptime: 00:03:14:214 > > >> > > > > > > > >> > > > >> > > > > > > > >> > > > >> > > > > > > > >> > [21:25:16,252][INFO > > >> ][Thread-4][GridProjectionSelfTest1] > > >> > > > > > > > >> > > > >> > > > > > > > >> > >>> > > >> > +-----------------------------------------------------+ > > >> > > > > > > > >> > >>> Ignite ver. 1.1.1-SNAPSHOT#19700101-sha1:DEV > > >> stopped > > >> > OK > > >> > > > > > > > >> > >>> > > >> > +-----------------------------------------------------+ > > >> > > > > > > > >> > >>> Grid name: internal.GridProjectionSelfTest1 > > >> > > > > > > > >> > >>> Grid uptime: 00:03:14:916 > > >> > > > > > > > >> > > > >> > > > > > > > >> > > > >> > > > > > > > >> > [21:25:16,266][INFO > > >> ][Thread-6][GridProjectionSelfTest2] > > >> > > > > > > > >> > > > >> > > > > > > > >> > >>> > > >> > +-----------------------------------------------------+ > > >> > > > > > > > >> > >>> Ignite ver. 1.1.1-SNAPSHOT#19700101-sha1:DEV > > >> stopped > > >> > OK > > >> > > > > > > > >> > >>> > > >> > +-----------------------------------------------------+ > > >> > > > > > > > >> > >>> Grid name: internal.GridProjectionSelfTest2 > > >> > > > > > > > >> > >>> Grid uptime: 00:03:14:618 > > >> > > > > > > > >> > > > >> > > > > > > > >> > > > >> > > > > > > > >> > which I assume means successful run. > > >> > > > > > > > >> > > > >> > > > > > > > >> > I am sorry about the formatting issues. For me > > getting > > >> the > > >> > > > > > > formatting > > >> > > > > > > > in > > >> > > > > > > > >> > place is always the biggest problem :) > > >> > > > > > > > >> > > > >> > > > > > > > >> > I have learnt a lot about the ASF coding formats > and > > >> > general > > >> > > > > Java > > >> > > > > > > > >> codelines > > >> > > > > > > > >> > (this is the first time I have written Java in my > > >> life). > > >> > So > > >> > > I > > >> > > > am > > >> > > > > > > sure > > >> > > > > > > > >> that > > >> > > > > > > > >> > this patch has allowed me to learn the ways to be > > used > > >> in > > >> > > > > further > > >> > > > > > > > >> patches > > >> > > > > > > > >> > that I write and review. > > >> > > > > > > > >> > > > >> > > > > > > > >> > On Mon, Jun 8, 2015 at 9:34 PM, Artiom Shutak < > > >> > > > > > ashu...@gridgain.com > > >> > > > > > > > > > >> > > > > > > > >> > wrote: > > >> > > > > > > > >> > > > >> > > > > > > > >> > > As I can see by history for new failed tests they > > are > > >> > not > > >> > > > > stable > > >> > > > > > > and > > >> > > > > > > > >> fail > > >> > > > > > > > >> > > sometimes. > > >> > > > > > > > >> > > > > >> > > > > > > > >> > > I suggest you to focus on your issue and your new > > >> test > > >> > at > > >> > > > > first. > > >> > > > > > > > >> > > > > >> > > > > > > > >> > > -- Artem -- > > >> > > > > > > > >> > > > > >> > > > > > > > >> > > On Mon, Jun 8, 2015 at 6:52 PM, Atri Sharma < > > >> > > > > > atri.j...@gmail.com> > > >> > > > > > > > >> wrote: > > >> > > > > > > > >> > > > > >> > > > > > > > >> > > > Thanks for the advice. > > >> > > > > > > > >> > > > > > >> > > > > > > > >> > > > Actually I am a bit lost here. I can understand > > why > > >> > the > > >> > > > new > > >> > > > > > > test I > > >> > > > > > > > >> > added > > >> > > > > > > > >> > > > fails but I am not sure why the rest of the > tests > > >> fail > > >> > > > > since I > > >> > > > > > > > have > > >> > > > > > > > >> not > > >> > > > > > > > >> > > > touched any code that follows the call path for > > >> those > > >> > > > tests. > > >> > > > > > > Sorry > > >> > > > > > > > >> if I > > >> > > > > > > > >> > > am > > >> > > > > > > > >> > > > being naive, but is there a way I can identify > > >> what is > > >> > > > > > causing a > > >> > > > > > > > >> test > > >> > > > > > > > >> > to > > >> > > > > > > > >> > > > fail, please? > > >> > > > > > > > >> > > > > > >> > > > > > > > >> > > > On Mon, Jun 8, 2015 at 9:17 PM, Artiom Shutak < > > >> > > > > > > > ashu...@gridgain.com > > >> > > > > > > > >> > > > >> > > > > > > > >> > > > wrote: > > >> > > > > > > > >> > > > > > >> > > > > > > > >> > > > > To run tests locally use: > > >> > > > > > > > >> > > > > > > >> > > > > > > > >> > > > > mvn clean test -U > > >> > -Plgpl,examples,-clean-libs,-release > > >> > > > > > > > >> > > > > -Dmaven.test.failure.ignore=true > > >> > -DfailIfNoTests=false > > >> > > > > > > > >> > > > > -Dtest=%TEST_PATTERN% > > >> > > > > > > > >> > > > > > > >> > > > > > > > >> > > > > For example, %TEST_PATTERN% can be > > >> > > > > > > > >> > > > > > > >> 'org.apache.ignite.testsuites.IgniteBasicTestSuite' > > >> > or > > >> > > > > > > > >> > > > > 'GridCacheLocalAtomicFullApiSelfTest#testGet' > > >> > > > > > > > >> > > > > > > >> > > > > > > > >> > > > > In your case > > >> > > > > > > > >> > > > > > > >> > > > > > > > >> > > > > mvn clean test -U > > >> > -Plgpl,examples,-clean-libs,-release > > >> > > > > > > > >> > > > > -Dmaven.test.failure.ignore=true > > >> > -DfailIfNoTests=false > > >> > > > > > -Dtest= > > >> > > > > > > > >> > > > > GridProjectionSelfTest#testforHostString > > >> > > > > > > > >> > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > > > >> > > > > -- Artem -- > > >> > > > > > > > >> > > > > > > >> > > > > > > > >> > > > > On Mon, Jun 8, 2015 at 5:07 PM, Atri Sharma < > > >> > > > > > > > atri.j...@gmail.com> > > >> > > > > > > > >> > > wrote: > > >> > > > > > > > >> > > > > > > >> > > > > > > > >> > > > > > Can you advice on the test failures > please? I > > >> have > > >> > > no > > >> > > > > idea > > >> > > > > > > of > > >> > > > > > > > >> > > debugging > > >> > > > > > > > >> > > > > the > > >> > > > > > > > >> > > > > > plethora of tests but I suspect that the > same > > >> > fault > > >> > > in > > >> > > > > the > > >> > > > > > > > >> patch is > > >> > > > > > > > >> > > > > causing > > >> > > > > > > > >> > > > > > them to fail... > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > On Mon, Jun 8, 2015 at 7:18 PM, Artiom > > Shutak < > > >> > > > > > > > >> > ashu...@gridgain.com> > > >> > > > > > > > >> > > > > > wrote: > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > Atri, see my comments at Jira. > > >> > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > I want to ask you to be more careful with > > all > > >> > > review > > >> > > > > > > > comments > > >> > > > > > > > >> > > (reread > > >> > > > > > > > >> > > > > all > > >> > > > > > > > >> > > > > > > above) and fix all of them. > > >> > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > -- Artem -- > > >> > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > On Mon, Jun 8, 2015 at 1:49 PM, Atri > > Sharma < > > >> > > > > > > > >> atri.j...@gmail.com > > >> > > > > > > > >> > > > > >> > > > > > > > >> > > > > wrote: > > >> > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > > Fixed and attached. > > >> > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > > > Please see and let me know your > feedback > > >> and > > >> > > > > comments. > > >> > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > > > On Fri, Jun 5, 2015 at 4:44 PM, Artiom > > >> Shutak > > >> > < > > >> > > > > > > > >> > > > ashu...@gridgain.com> > > >> > > > > > > > >> > > > > > > > wrote: > > >> > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > > > > Hi Atri, > > >> > > > > > > > >> > > > > > > > > > > >> > > > > > > > >> > > > > > > > > See my new review in jira. > > >> > > > > > > > >> > > > > > > > > > > >> > > > > > > > >> > > > > > > > > -- Artem -- > > >> > > > > > > > >> > > > > > > > > > > >> > > > > > > > >> > > > > > > > > On Fri, Jun 5, 2015 at 11:20 AM, Atri > > >> > Sharma < > > >> > > > > > > > >> > > > atri.j...@gmail.com> > > >> > > > > > > > >> > > > > > > > wrote: > > >> > > > > > > > >> > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > Artem, > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > Thanks for your comments. > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > I incorporated your comments and > sent > > >> new > > >> > > > patch. > > >> > > > > > > > Please > > >> > > > > > > > >> see > > >> > > > > > > > >> > > and > > >> > > > > > > > >> > > > > > > > comment. > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > Regards, > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > Atri > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > On Fri, Jun 5, 2015 at 3:22 AM, > > Artiom > > >> > > Shutak > > >> > > > < > > >> > > > > > > > >> > > > > > ashu...@gridgain.com> > > >> > > > > > > > >> > > > > > > > > > wrote: > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > Atri, please, see my comments at > > the > > >> > Jira > > >> > > > > > ticket. > > >> > > > > > > > >> > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > -- Artem -- > > >> > > > > > > > >> > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > On Thu, Jun 4, 2015 at 7:44 PM, > > Atri > > >> > > Sharma > > >> > > > < > > >> > > > > > > > >> > > > > atri.j...@gmail.com > > >> > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > > > wrote: > > >> > > > > > > > >> > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > Thanks. > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > Sorry for the noise on this > one. > > I > > >> am > > >> > > off > > >> > > > > next > > >> > > > > > > > week > > >> > > > > > > > >> so > > >> > > > > > > > >> > > it's > > >> > > > > > > > >> > > > > > > helpful > > >> > > > > > > > >> > > > > > > > > if > > >> > > > > > > > >> > > > > > > > > > we > > >> > > > > > > > >> > > > > > > > > > > > can close this by this week if > > >> > > possible. I > > >> > > > > am > > >> > > > > > > > >> willing > > >> > > > > > > > >> > to > > >> > > > > > > > >> > > > work > > >> > > > > > > > >> > > > > > in > > >> > > > > > > > >> > > > > > > my > > >> > > > > > > > >> > > > > > > > > > > > midnight for that if needed :) > > >> > > > > > > > >> > > > > > > > > > > > On 4 Jun 2015 01:53, "Dmitriy > > >> > > Setrakyan" < > > >> > > > > > > > >> > > > > > dsetrak...@apache.org> > > >> > > > > > > > >> > > > > > > > > > wrote: > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > Atri, > > >> > > > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > Thanks for providing a patch > to > > >> > > Ignite! > > >> > > > I > > >> > > > > am > > >> > > > > > > > sure > > >> > > > > > > > >> > > someone > > >> > > > > > > > >> > > > > in > > >> > > > > > > > >> > > > > > > the > > >> > > > > > > > >> > > > > > > > > > > > community > > >> > > > > > > > >> > > > > > > > > > > > > will be looking into it > shortly > > >> (if > > >> > > not > > >> > > > > > > > already). > > >> > > > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > Once your patch is accepted, > > you > > >> > will > > >> > > be > > >> > > > > > > > eligible > > >> > > > > > > > >> to > > >> > > > > > > > >> > > > > become a > > >> > > > > > > > >> > > > > > > > > > > committer. > > >> > > > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > D. > > >> > > > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > On Tue, Jun 2, 2015 at 7:03 > AM, > > >> Atri > > >> > > > > Sharma > > >> > > > > > < > > >> > > > > > > > >> > > > > > > atri.j...@gmail.com > > >> > > > > > > > >> > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > wrote: > > >> > > > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > Patch version 1 is > available. > > >> > Please > > >> > > > see > > >> > > > > > and > > >> > > > > > > > >> > comment. > > >> > > > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > On Mon, Jun 1, 2015 at 9:18 > > PM, > > >> > Atri > > >> > > > > > Sharma > > >> > > > > > > < > > >> > > > > > > > >> > > > > > > > atri.j...@gmail.com > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > wrote: > > >> > > > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > Thanks. > > >> > > > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > On Mon, Jun 1, 2015 at > 9:16 > > >> PM, > > >> > > > Yakov > > >> > > > > > > > Zhdanov > > >> > > > > > > > >> < > > >> > > > > > > > >> > > > > > > > > > yzhda...@apache.org > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > wrote: > > >> > > > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> Did you see the comment > in > > >> the > > >> > > > > ticket? > > >> > > > > > > > Andrey > > >> > > > > > > > >> > Gura > > >> > > > > > > > >> > > > has > > >> > > > > > > > >> > > > > > put > > >> > > > > > > > >> > > > > > > > > some > > >> > > > > > > > >> > > > > > > > > > > tips > > >> > > > > > > > >> > > > > > > > > > > > > > >> there. > > >> > > > > > > > >> > > > > > > > > > > > > > >> You can ask further > > >> questions > > >> > in > > >> > > > the > > >> > > > > > > > ticket. > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > >> > > > > > > > >> > > > > > > > > > > > > > >> Good luck! > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > >> > > > > > > > >> > > > > > > > > > > > > > >> --Yakov > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > >> > > > > > > > >> > > > > > > > > > > > > > >> 2015-06-01 18:40 > GMT+03:00 > > >> Atri > > >> > > > > Sharma > > >> > > > > > < > > >> > > > > > > > >> > > > > > > atri.j...@gmail.com > > >> > > > > > > > >> > > > > > > > >: > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > >> > > > > > > > >> > > > > > > > > > > > > > >> > Thanks Yakov. > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > Can you also please > > >> advice at > > >> > > > code > > >> > > > > > > areas > > >> > > > > > > > >> > where I > > >> > > > > > > > >> > > > > > should > > >> > > > > > > > >> > > > > > > > look > > >> > > > > > > > >> > > > > > > > > > at > > >> > > > > > > > >> > > > > > > > > > > > for > > >> > > > > > > > >> > > > > > > > > > > > > > this > > >> > > > > > > > >> > > > > > > > > > > > > > >> > please? > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > On Mon, Jun 1, 2015 at > > >> 9:03 > > >> > PM, > > >> > > > > Yakov > > >> > > > > > > > >> Zhdanov > > >> > > > > > > > >> > < > > >> > > > > > > > >> > > > > > > > > > > > yzhda...@apache.org> > > >> > > > > > > > >> > > > > > > > > > > > > > >> wrote: > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > Arti you can read > > about > > >> > > sprint > > >> > > > > > > process > > >> > > > > > > > >> here > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > >> > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Sprint+Process > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > This page will be > soon > > >> > > updated > > >> > > > > with > > >> > > > > > > > >> > > information > > >> > > > > > > > >> > > > on > > >> > > > > > > > >> > > > > > how > > >> > > > > > > > >> > > > > > > > to > > >> > > > > > > > >> > > > > > > > > > > format > > >> > > > > > > > >> > > > > > > > > > > > > and > > >> > > > > > > > >> > > > > > > > > > > > > > >> > submit > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > patch and check CI > > >> tests. > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > --Yakov > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > 2015-06-01 12:54 > > >> GMT+03:00 > > >> > > Atri > > >> > > > > > > Sharma > > >> > > > > > > > < > > >> > > > > > > > >> > > > > > > > > atri.j...@gmail.com > > >> > > > > > > > >> > > > > > > > > > >: > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > I would like to > work > > >> on > > >> > > 917, > > >> > > > > have > > >> > > > > > > > >> assigned > > >> > > > > > > > >> > > it > > >> > > > > > > > >> > > > to > > >> > > > > > > > >> > > > > > > > myself. > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > Please advice on > the > > >> > > approach > > >> > > > > and > > >> > > > > > > > code > > >> > > > > > > > >> > areas > > >> > > > > > > > >> > > > > this > > >> > > > > > > > >> > > > > > > > patch > > >> > > > > > > > >> > > > > > > > > > will > > >> > > > > > > > >> > > > > > > > > > > > > need > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > modifications in. > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > -- > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > Regards, > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > Atri > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > *l'apprenant* > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > -- > > >> > > > > > > > >> > > > > > > > > > > > > > >> > Regards, > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > Atri > > >> > > > > > > > >> > > > > > > > > > > > > > >> > *l'apprenant* > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > >> > > > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > -- > > >> > > > > > > > >> > > > > > > > > > > > > > > Regards, > > >> > > > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > Atri > > >> > > > > > > > >> > > > > > > > > > > > > > > *l'apprenant* > > >> > > > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > -- > > >> > > > > > > > >> > > > > > > > > > > > > > Regards, > > >> > > > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > Atri > > >> > > > > > > > >> > > > > > > > > > > > > > *l'apprenant* > > >> > > > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > -- > > >> > > > > > > > >> > > > > > > > > > Regards, > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > Atri > > >> > > > > > > > >> > > > > > > > > > *l'apprenant* > > >> > > > > > > > >> > > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > >> > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > > > -- > > >> > > > > > > > >> > > > > > > > Regards, > > >> > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > > > Atri > > >> > > > > > > > >> > > > > > > > *l'apprenant* > > >> > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > -- > > >> > > > > > > > >> > > > > > Regards, > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > Atri > > >> > > > > > > > >> > > > > > *l'apprenant* > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > > > >> > > > > > >> > > > > > > > >> > > > > > >> > > > > > > > >> > > > > > >> > > > > > > > >> > > > -- > > >> > > > > > > > >> > > > Regards, > > >> > > > > > > > >> > > > > > >> > > > > > > > >> > > > Atri > > >> > > > > > > > >> > > > *l'apprenant* > > >> > > > > > > > >> > > > > > >> > > > > > > > >> > > > > >> > > > > > > > >> > > > >> > > > > > > > >> > > > >> > > > > > > > >> > > > >> > > > > > > > >> > -- > > >> > > > > > > > >> > Regards, > > >> > > > > > > > >> > > > >> > > > > > > > >> > Atri > > >> > > > > > > > >> > *l'apprenant* > > >> > > > > > > > >> > > > >> > > > > > > > >> > > >> > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > -- > > >> > > > > > > > > Regards, > > >> > > > > > > > > > > >> > > > > > > > > Atri > > >> > > > > > > > > *l'apprenant* > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > -- > > >> > > > > > Regards, > > >> > > > > > > > >> > > > > > Atri > > >> > > > > > *l'apprenant* > > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > >> > > > >> > > > >> > -- > > >> > Regards, > > >> > > > >> > Atri > > >> > *l'apprenant* > > >> > > > >> > > > > > > > > > > > > -- > > > Regards, > > > > > > Atri > > > *l'apprenant* > > > > > >
