You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@ignite.apache.org by Saikat Maitra <sa...@gmail.com> on 2016/10/25 19:51:20 UTC

IGNITE-4103 IgniteDataStreamer doesn't throw exception if cache is destroyed

Hi,

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/1186

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

Re: IGNITE-4103 IgniteDataStreamer doesn't throw exception if cache is destroyed

Posted by Valentin Kulichenko <va...@gmail.com>.

Hi Saikat,

Sorry, I missed your comment in the ticket. Will take a look!

-Val

On Sun, Nov 13, 2016 at 12:54 AM, Saikat Maitra <sa...@gmail.com>
wrote:

> Hi Valentin,
>
> I have updated the PR after refactoring the files. There are changes in 20
> files including the tests.
>
> Please review the changes and let me know if any further changes is
> required.
>
> Regards,
> Saikat
>
> On Wed, Oct 26, 2016 at 11:24 PM, Saikat Maitra <sa...@gmail.com>
> wrote:
>
> > Thank you Valentin !!!
> >
> > Regards
> > Saikat
> >
> > On Wed, Oct 26, 2016 at 3:38 AM, Valentin Kulichenko <
> > valentin.kulichenko@gmail.com> wrote:
> >
> >> Hi Saikat,
> >>
> >> I added a comment in the ticket.
> >>
> >> -Val
> >>
> >> On Tue, Oct 25, 2016 at 12:51 PM, Saikat Maitra <
> saikat.maitra@gmail.com>
> >> wrote:
> >>
> >> > Hi,
> >> >
> >> > 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/1186
> >> >
> >> > [2] https://issues.apache.org/jira/browse/IGNITE-4103
> >> >
> >>
> >
> >
>

Re: IGNITE-4103 IgniteDataStreamer doesn't throw exception if cache is destroyed

Posted by Saikat Maitra <sa...@gmail.com>.

Hi Valentin,

I have updated the PR after refactoring the files. There are changes in 20
files including the tests.

Please review the changes and let me know if any further changes is
required.

Regards,
Saikat

On Wed, Oct 26, 2016 at 11:24 PM, Saikat Maitra <sa...@gmail.com>
wrote:

> Thank you Valentin !!!
>
> Regards
> Saikat
>
> On Wed, Oct 26, 2016 at 3:38 AM, Valentin Kulichenko <
> valentin.kulichenko@gmail.com> wrote:
>
>> Hi Saikat,
>>
>> I added a comment in the ticket.
>>
>> -Val
>>
>> On Tue, Oct 25, 2016 at 12:51 PM, Saikat Maitra <sa...@gmail.com>
>> wrote:
>>
>> > Hi,
>> >
>> > 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/1186
>> >
>> > [2] https://issues.apache.org/jira/browse/IGNITE-4103
>> >
>>
>
>

Re: IGNITE-4103 IgniteDataStreamer doesn't throw exception if cache is destroyed

Posted by Saikat Maitra <sa...@gmail.com>.

Thank you Valentin !!!

Regards
Saikat

On Wed, Oct 26, 2016 at 3:38 AM, Valentin Kulichenko <
valentin.kulichenko@gmail.com> wrote:

> Hi Saikat,
>
> I added a comment in the ticket.
>
> -Val
>
> On Tue, Oct 25, 2016 at 12:51 PM, Saikat Maitra <sa...@gmail.com>
> wrote:
>
> > Hi,
> >
> > 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/1186
> >
> > [2] https://issues.apache.org/jira/browse/IGNITE-4103
> >
>

Re: IGNITE-4103 IgniteDataStreamer doesn't throw exception if cache is destroyed

Posted by Valentin Kulichenko <va...@gmail.com>.

Hi Saikat,

I added a comment in the ticket.

-Val

On Tue, Oct 25, 2016 at 12:51 PM, Saikat Maitra <sa...@gmail.com>
wrote:

> Hi,
>
> 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/1186
>
> [2] https://issues.apache.org/jira/browse/IGNITE-4103
>