Apache Ignite Developers - Legacy Mail Archive

IGNITE-3841 Web console: we need to add eviction policy consistency

Classic

List

Threaded

6 messages Options

Saikat Maitra-2

IGNITE-3841 Web console: we need to add eviction policy consistency

Hello,

I have raised PR[1] for the following jira ticket[2].

Please review and let me know if any changes required.

Regards,

Saikat

[1] https://github.com/apache/ignite/pull/1136
<https://github.com/apache/ignite/pull/1112>

[2] https://issues.apache.org/jira/browse/IGNITE-3841
<https://issues.apache.org/jira/browse/IGNITE-3963>

Saikat Maitra-2

Re: IGNITE-3841 Web console: we need to add eviction policy consistency

Hi

The hyperlinks in the above thread points to incorrect PR. I have shared
the correct urls below.

[1] https://github.com/apache/ignite/pull/1136

[2] https://issues.apache.org/jira/browse/IGNITE-3841

Regards

Saikat

On Sun, Oct 2, 2016 at 12:08 AM, Saikat Maitra <[hidden email]> wrote:

> Hello,
>
> I have raised PR[1] for the following jira ticket[2].
>
> Please review and let me know if any changes required.
>
> Regards,
>
> Saikat
>
> [1] https://github.com/apache/ignite/pull/1136
> <https://github.com/apache/ignite/pull/1112>
>
> [2] https://issues.apache.org/jira/browse/IGNITE-3841
> <https://issues.apache.org/jira/browse/IGNITE-3963>
>

Denis Magda

Re: IGNITE-3841 Web console: we need to add eviction policy consistency

Igniters,

Who will be able to review Saikat’s changes?

—
Denis

> On Oct 1, 2016, at 12:42 PM, Saikat Maitra <[hidden email]> wrote:
>
> Hi
>
> The hyperlinks in the above thread points to incorrect PR. I have shared
> the correct urls below.
>
> [1] https://github.com/apache/ignite/pull/1136
>
> [2] https://issues.apache.org/jira/browse/IGNITE-3841
>
>
> Regards
>
> Saikat
>
> On Sun, Oct 2, 2016 at 12:08 AM, Saikat Maitra <[hidden email]> wrote:
>
>> Hello,
>>
>> I have raised PR[1] for the following jira ticket[2].
>>
>> Please review and let me know if any changes required.
>>
>> Regards,
>>
>> Saikat
>>
>> [1] https://github.com/apache/ignite/pull/1136
>> <https://github.com/apache/ignite/pull/1112>
>>
>> [2] https://issues.apache.org/jira/browse/IGNITE-3841
>> <https://issues.apache.org/jira/browse/IGNITE-3963>
>>

Alexey Kuznetsov-2

Re: IGNITE-3841 Web console: we need to add eviction policy consistency

I will review them tomorrow.

On Mon, Oct 3, 2016 at 11:15 PM, Denis Magda <[hidden email]> wrote:

> Igniters,
>
> Who will be able to review Saikat’s changes?
>
> —
> Denis
>
> > On Oct 1, 2016, at 12:42 PM, Saikat Maitra <[hidden email]> wrote:
> >
> > Hi
> >
> > The hyperlinks in the above thread points to incorrect PR. I have shared
> > the correct urls below.
> >
> > [1] https://github.com/apache/ignite/pull/1136
> >
> > [2] https://issues.apache.org/jira/browse/IGNITE-3841
> >
> >
> > Regards
> >
> > Saikat
> >
> > On Sun, Oct 2, 2016 at 12:08 AM, Saikat Maitra <[hidden email]>
> wrote:
> >
> >> Hello,
> >>
> >> I have raised PR[1] for the following jira ticket[2].
> >>
> >> Please review and let me know if any changes required.
> >>
> >> Regards,
> >>
> >> Saikat
> >>
> >> [1] https://github.com/apache/ignite/pull/1136
> >> <https://github.com/apache/ignite/pull/1112>
> >>
> >> [2] https://issues.apache.org/jira/browse/IGNITE-3841
> >> <https://issues.apache.org/jira/browse/IGNITE-3963>
> >>
>
>

--
Alexey Kuznetsov
GridGain Systems
www.gridgain.com

Alexey Kuznetsov-2

Re: IGNITE-3841 Web console: we need to add eviction policy consistency

In reply to this post by Saikat Maitra-2

Saikat,

I reviewed your PR. Changed it a little and merged to master.
Thanks for your contribution!

--
Alexey Kuznetsov

Saikat Maitra

Re: IGNITE-3841 Web console: we need to add eviction policy consistency

Thank you Alexey !!!

Regards
Saikat

On Tue, Oct 4, 2016 at 3:32 PM, Alexey Kuznetsov <[hidden email]>
wrote:

> Saikat,
>
> I reviewed your PR. Changed it a little and merged to master.
> Thanks for your contribution!
>
> --
> Alexey Kuznetsov
>