ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Atri Sharma <atri.j...@gmail.com>
Subject Re: IGNITE 917
Date Mon, 22 Jun 2015 18:20:21 GMT
Guys,

Did Sprint -7 happen today? Just checking

On Fri, Jun 19, 2015 at 11:52 PM, Atri Sharma <atri.jiit@gmail.com> wrote:

> Thanks artem.
>
> Sorry,  was not aware of the commit schedule policy
> On 19 Jun 2015 23:39, "Artiom Shutak" <ashutak@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.jiit@gmail.com> wrote:
>>
>> > Folks,
>> >
>> > Any update on this please?
>> > On 11 Jun 2015 15:14, "Atri Sharma" <atri.jiit@gmail.com> wrote:
>> >
>> > > That makes sense, thanks!
>> > >
>> > > Thanks for your review!
>> > >
>> > > On Thu, Jun 11, 2015 at 3:13 PM, Artiom Shutak <ashutak@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.jiit@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 <
>> ashutak@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.jiit@gmail.com>
>> > >> > wrote:
>> > >> > >
>> > >> > > > Please let me know your comments and feedback.
>> > >> > > > On 9 Jun 2015 23:38, "Artiom Shutak" <ashutak@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.jiit@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 <
>> > >> > ashutak@gridgain.com
>> > >> > > >
>> > >> > > > > > wrote:
>> > >> > > > > >
>> > >> > > > > > > See comments at Jira.
>> > >> > > > > > >
>> > >> > > > > > > -- Artem --
>> > >> > > > > > >
>> > >> > > > > > > On Mon, Jun 8, 2015 at 8:21 PM, Atri Sharma <
>> > >> atri.jiit@gmail.com
>> > >> > >
>> > >> > > > > wrote:
>> > >> > > > > > >
>> > >> > > > > > > > Latest patch uploaded.  Please see and let me know your
>> > >> > comments
>> > >> > > > > > > > On 8 Jun 2015 22:19, "Atri Sharma" <
>> atri.jiit@gmail.com>
>> > >> > wrote:
>> > >> > > > > > > >
>> > >> > > > > > > > > Thanks for that.
>> > >> > > > > > > > >
>> > >> > > > > > > > > So got a build success.
>> > >> > > > > > > > >
>> > >> > > > > > > > > On Mon, Jun 8, 2015 at 10:07 PM, Artiom Shutak <
>> > >> > > > > ashutak@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.jiit@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 <
>> > >> > > > > > ashutak@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.jiit@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
>> <
>> > >> > > > > > > > ashutak@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.jiit@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 <
>> > >> > > > > > > > >> > ashutak@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.jiit@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
>> > >> > <
>> > >> > > > > > > > >> > > > ashutak@gridgain.com>
>> > >> > > > > > > > >> > > > > > > > wrote:
>> > >> > > > > > > > >> > > > > > > >
>> > >> > > > > > > > >> > > > > > > > > Hi Atri,
>> > >> > > > > > > > >> > > > > > > > >
>> > >> > > > > > > > >> > > > > > > > > See my new review in jira.
>> > >> > > > > > > > >> > > > > > > > >
>> > >> > > > > > > > >> > > > > > > > > -- Artem --
>> > >> > > > > > > > >> > > > > > > > >
>> > >> > > > > > > > >> > > > > > > > > On Fri, Jun 5, 2015 at 11:20 AM,
>> Atri
>> > >> > Sharma <
>> > >> > > > > > > > >> > > > atri.jiit@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
>> > >> > > > <
>> > >> > > > > > > > >> > > > > > ashutak@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.jiit@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" <
>> > >> > > > > > > > >> > > > > > dsetrakyan@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.jiit@gmail.com
>> > >> > > > > > > > >> > > > > > > > >
>> > >> > > > > > > > >> > > > > > > > > > > wrote:
>> > >> > > > > > > > >> > > > > > > > > > > > >
>> > >> > > > > > > > >> > > > > > > > > > > > > > Patch version 1 is
>> available.
>> > >> > Please
>> > >> > > > see
>> > >> > > > > > and
>> > >> > > > > > > > >> > comment.
>> > >> > > > > > > > >> > > > > > > > > > > > > >
>> > >> > > > > > > > >> > > > > > > > > > > > > > On Mon, Jun 1, 2015 at
>> 9:18
>> > PM,
>> > >> > Atri
>> > >> > > > > > Sharma
>> > >> > > > > > > <
>> > >> > > > > > > > >> > > > > > > > atri.jiit@gmail.com
>> > >> > > > > > > > >> > > > > > > > > >
>> > >> > > > > > > > >> > > > > > > > > > > > wrote:
>> > >> > > > > > > > >> > > > > > > > > > > > > >
>> > >> > > > > > > > >> > > > > > > > > > > > > > > Thanks.
>> > >> > > > > > > > >> > > > > > > > > > > > > > >
>> > >> > > > > > > > >> > > > > > > > > > > > > > > On Mon, Jun 1, 2015 at
>> 9:16
>> > >> PM,
>> > >> > > > Yakov
>> > >> > > > > > > > Zhdanov
>> > >> > > > > > > > >> <
>> > >> > > > > > > > >> > > > > > > > > > yzhdanov@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.jiit@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
>> > >> > > > > > > > >> > <
>> > >> > > > > > > > >> > > > > > > > > > > > yzhdanov@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.jiit@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*
>> > >
>> >
>>
>


-- 
Regards,

Atri
*l'apprenant*

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message