Apache Ignite Developers - Legacy Mail Archive

IGNITE 1055

Classic

List

Threaded

11 messages Options

Atri Sharma

IGNITE 1055

Folks,

I have implemented first version of patch. Patch is uploaded in ticket.

Please see and let me know your comments and feedback.

Regards,

Atri

--
Regards,

Atri
*l'apprenant*

Atri Sharma

Re: IGNITE 1055

Folks,

Any chance we can get this in sprint 7 please?
On 26 Jun 2015 21:19, "Atri Sharma" <[hidden email]> wrote:

> Folks,
>
> I have implemented first version of patch. Patch is uploaded in ticket.
>
> Please see and let me know your comments and feedback.
>
> Regards,
>
> Atri
>
> --
> Regards,
>
> Atri
> *l'apprenant*
>

Artem Shutak

Re: IGNITE 1055

Hi, Atri,

I've looked at your patch and I'm ok with it in general. I have a small
comment (see jira).

-- Artem --

On Mon, Jun 29, 2015 at 12:58 PM, Atri Sharma <[hidden email]> wrote:

> Folks,
>
> Any chance we can get this in sprint 7 please?
> On 26 Jun 2015 21:19, "Atri Sharma" <[hidden email]> wrote:
>
> > Folks,
> >
> > I have implemented first version of patch. Patch is uploaded in ticket.
> >
> > Please see and let me know your comments and feedback.
> >
> > Regards,
> >
> > Atri
> >
> > --
> > Regards,
> >
> > Atri
> > *l'apprenant*
> >
>

Atri Sharma

Re: IGNITE 1055

Thank you so much.
Let me fix and push

On Mon, Jun 29, 2015 at 8:40 PM, Artiom Shutak <[hidden email]> wrote:

> Hi, Atri,
>
> I've looked at your patch and I'm ok with it in general. I have a small
> comment (see jira).
>
>
> -- Artem --
>
> On Mon, Jun 29, 2015 at 12:58 PM, Atri Sharma <[hidden email]> wrote:
>
> > Folks,
> >
> > Any chance we can get this in sprint 7 please?
> > On 26 Jun 2015 21:19, "Atri Sharma" <[hidden email]> wrote:
> >
> > > Folks,
> > >
> > > I have implemented first version of patch. Patch is uploaded in ticket.
> > >
> > > Please see and let me know your comments and feedback.
> > >
> > > Regards,
> > >
> > > Atri
> > >
> > > --
> > > Regards,
> > >
> > > Atri
> > > *l'apprenant*
> > >
> >
>

--
Regards,

Atri
*l'apprenant*

Atri Sharma

Re: IGNITE 1055

In reply to this post by Artem Shutak

Hi,

I have made changes and uploaded patch in JIRA.

Please see and let me know your comments and feedback.

Regards,

Atri

On Mon, Jun 29, 2015 at 8:40 PM, Artiom Shutak <[hidden email]> wrote:

--
Regards,

Atri
*l'apprenant*

Atri Sharma

Re: IGNITE 1055

In reply to this post by Artem Shutak

Hi Artem,

If the patch is ok, can you please merge it in sprint 7 if its not too much
of bothering?

Thanks and Regards,

Atri

On Mon, Jun 29, 2015 at 8:40 PM, Artiom Shutak <[hidden email]> wrote:

--
Regards,

Atri
*l'apprenant*

Artem Shutak

Re: IGNITE 1055

I think someone from commiters will look at patch (approximately tomorrow)
and merge it if it's okey.

-- Artem --

On Tue, Jun 30, 2015 at 6:13 PM, Atri Sharma <[hidden email]> wrote:

> Hi Artem,
>
> If the patch is ok, can you please merge it in sprint 7 if its not too much
> of bothering?
>
> Thanks and Regards,
>
> Atri
>
> On Mon, Jun 29, 2015 at 8:40 PM, Artiom Shutak <[hidden email]>
> wrote:
>
> > Hi, Atri,
> >
> > I've looked at your patch and I'm ok with it in general. I have a small
> > comment (see jira).
> >
> >
> > -- Artem --
> >
> > On Mon, Jun 29, 2015 at 12:58 PM, Atri Sharma <[hidden email]>
> wrote:
> >
> > > Folks,
> > >
> > > Any chance we can get this in sprint 7 please?
> > > On 26 Jun 2015 21:19, "Atri Sharma" <[hidden email]> wrote:
> > >
> > > > Folks,
> > > >
> > > > I have implemented first version of patch. Patch is uploaded in
> ticket.
> > > >
> > > > Please see and let me know your comments and feedback.
> > > >
> > > > Regards,
> > > >
> > > > Atri
> > > >
> > > > --
> > > > Regards,
> > > >
> > > > Atri
> > > > *l'apprenant*
> > > >
> > >
> >
>
>
>
> --
> Regards,
>
> Atri
> *l'apprenant*
>

Atri Sharma

Re: IGNITE 1055

Thank you.

Please let me know when it is committed.

Regards,

Atri
On 30 Jun 2015 23:38, "Artiom Shutak" <[hidden email]> wrote:

> I think someone from commiters will look at patch (approximately tomorrow)
> and merge it if it's okey.
>
> -- Artem --
>
> On Tue, Jun 30, 2015 at 6:13 PM, Atri Sharma <[hidden email]> wrote:
>
> > Hi Artem,
> >
> > If the patch is ok, can you please merge it in sprint 7 if its not too
> much
> > of bothering?
> >
> > Thanks and Regards,
> >
> > Atri
> >
> > On Mon, Jun 29, 2015 at 8:40 PM, Artiom Shutak <[hidden email]>
> > wrote:
> >
> > > Hi, Atri,
> > >
> > > I've looked at your patch and I'm ok with it in general. I have a small
> > > comment (see jira).
> > >
> > >
> > > -- Artem --
> > >
> > > On Mon, Jun 29, 2015 at 12:58 PM, Atri Sharma <[hidden email]>
> > wrote:
> > >
> > > > Folks,
> > > >
> > > > Any chance we can get this in sprint 7 please?
> > > > On 26 Jun 2015 21:19, "Atri Sharma" <[hidden email]> wrote:
> > > >
> > > > > Folks,
> > > > >
> > > > > I have implemented first version of patch. Patch is uploaded in
> > ticket.
> > > > >
> > > > > Please see and let me know your comments and feedback.
> > > > >
> > > > > Regards,
> > > > >
> > > > > Atri
> > > > >
> > > > > --
> > > > > Regards,
> > > > >
> > > > > Atri
> > > > > *l'apprenant*
> > > > >
> > > >
> > >
> >
> >
> >
> > --
> > Regards,
> >
> > Atri
> > *l'apprenant*
> >
>

Atri Sharma

Re: IGNITE 1055

In reply to this post by Artem Shutak

Hi,

Did it get committed please?

Regards,

Atri

On Tue, Jun 30, 2015 at 11:37 PM, Artiom Shutak <[hidden email]>
wrote:

--
Regards,

Atri
*l'apprenant*

Artem Shutak

Re: IGNITE 1055

Hi,

ignite-1055 has been merged at sprint-7.

I've added new -nojmx option to 'ignite -help' and tested it on
linux/mac/windows.

-- Artem --

On Fri, Jul 3, 2015 at 2:29 PM, Atri Sharma <[hidden email]> wrote:

> Hi,
>
> Did it get committed please?
>
> Regards,
>
> Atri
>
> On Tue, Jun 30, 2015 at 11:37 PM, Artiom Shutak <[hidden email]>
> wrote:
>
> > I think someone from commiters will look at patch (approximately
> tomorrow)
> > and merge it if it's okey.
> >
> > -- Artem --
> >
> > On Tue, Jun 30, 2015 at 6:13 PM, Atri Sharma <[hidden email]>
> wrote:
> >
> > > Hi Artem,
> > >
> > > If the patch is ok, can you please merge it in sprint 7 if its not too
> > much
> > > of bothering?
> > >
> > > Thanks and Regards,
> > >
> > > Atri
> > >
> > > On Mon, Jun 29, 2015 at 8:40 PM, Artiom Shutak <[hidden email]>
> > > wrote:
> > >
> > > > Hi, Atri,
> > > >
> > > > I've looked at your patch and I'm ok with it in general. I have a
> small
> > > > comment (see jira).
> > > >
> > > >
> > > > -- Artem --
> > > >
> > > > On Mon, Jun 29, 2015 at 12:58 PM, Atri Sharma <[hidden email]>
> > > wrote:
> > > >
> > > > > Folks,
> > > > >
> > > > > Any chance we can get this in sprint 7 please?
> > > > > On 26 Jun 2015 21:19, "Atri Sharma" <[hidden email]> wrote:
> > > > >
> > > > > > Folks,
> > > > > >
> > > > > > I have implemented first version of patch. Patch is uploaded in
> > > ticket.
> > > > > >
> > > > > > Please see and let me know your comments and feedback.
> > > > > >
> > > > > > Regards,
> > > > > >
> > > > > > Atri
> > > > > >
> > > > > > --
> > > > > > Regards,
> > > > > >
> > > > > > Atri
> > > > > > *l'apprenant*
> > > > > >
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Regards,
> > >
> > > Atri
> > > *l'apprenant*
> > >
> >
>
>
>
> --
> Regards,
>
> Atri
> *l'apprenant*
>

Atri Sharma

Re: IGNITE 1055

Thank you so much!

On Fri, Jul 3, 2015 at 8:26 PM, Artiom Shutak <[hidden email]> wrote:

> Hi,
>
> ignite-1055 has been merged at sprint-7.
>
> I've added new -nojmx option to 'ignite -help' and tested it on
> linux/mac/windows.
>
> -- Artem --
>
> On Fri, Jul 3, 2015 at 2:29 PM, Atri Sharma <[hidden email]> wrote:
>
> > Hi,
> >
> > Did it get committed please?
> >
> > Regards,
> >
> > Atri
> >
> > On Tue, Jun 30, 2015 at 11:37 PM, Artiom Shutak <[hidden email]>
> > wrote:
> >
> > > I think someone from commiters will look at patch (approximately
> > tomorrow)
> > > and merge it if it's okey.
> > >
> > > -- Artem --
> > >
> > > On Tue, Jun 30, 2015 at 6:13 PM, Atri Sharma <[hidden email]>
> > wrote:
> > >
> > > > Hi Artem,
> > > >
> > > > If the patch is ok, can you please merge it in sprint 7 if its not
> too
> > > much
> > > > of bothering?
> > > >
> > > > Thanks and Regards,
> > > >
> > > > Atri
> > > >
> > > > On Mon, Jun 29, 2015 at 8:40 PM, Artiom Shutak <[hidden email]
> >
> > > > wrote:
> > > >
> > > > > Hi, Atri,
> > > > >
> > > > > I've looked at your patch and I'm ok with it in general. I have a
> > small
> > > > > comment (see jira).
> > > > >
> > > > >
> > > > > -- Artem --
> > > > >
> > > > > On Mon, Jun 29, 2015 at 12:58 PM, Atri Sharma <[hidden email]
> >
> > > > wrote:
> > > > >
> > > > > > Folks,
> > > > > >
> > > > > > Any chance we can get this in sprint 7 please?
> > > > > > On 26 Jun 2015 21:19, "Atri Sharma" <[hidden email]> wrote:
> > > > > >
> > > > > > > Folks,
> > > > > > >
> > > > > > > I have implemented first version of patch. Patch is uploaded in
> > > > ticket.
> > > > > > >
> > > > > > > Please see and let me know your comments and feedback.
> > > > > > >
> > > > > > > Regards,
> > > > > > >
> > > > > > > Atri
> > > > > > >
> > > > > > > --
> > > > > > > Regards,
> > > > > > >
> > > > > > > Atri
> > > > > > > *l'apprenant*
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Regards,
> > > >
> > > > Atri
> > > > *l'apprenant*
> > > >
> > >
> >
> >
> >
> > --
> > Regards,
> >
> > Atri
> > *l'apprenant*
> >
>

--
Regards,

Atri
*l'apprenant*