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

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

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

Saikat Maitra
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
Reply | Threaded
Open this post in threaded view
|

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

Valentin Kulichenko
Hi Saikat,

I added a comment in the ticket.

-Val

On Tue, Oct 25, 2016 at 12:51 PM, Saikat Maitra <[hidden email]>
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
>
Reply | Threaded
Open this post in threaded view
|

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

Saikat Maitra
Thank you Valentin !!!

Regards
Saikat

On Wed, Oct 26, 2016 at 3:38 AM, Valentin Kulichenko <
[hidden email]> wrote:

> Hi Saikat,
>
> I added a comment in the ticket.
>
> -Val
>
> On Tue, Oct 25, 2016 at 12:51 PM, Saikat Maitra <[hidden email]>
> 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
> >
>
Reply | Threaded
Open this post in threaded view
|

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

Saikat Maitra
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 <[hidden email]>
wrote:

> Thank you Valentin !!!
>
> Regards
> Saikat
>
> On Wed, Oct 26, 2016 at 3:38 AM, Valentin Kulichenko <
> [hidden email]> wrote:
>
>> Hi Saikat,
>>
>> I added a comment in the ticket.
>>
>> -Val
>>
>> On Tue, Oct 25, 2016 at 12:51 PM, Saikat Maitra <[hidden email]>
>> 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
>> >
>>
>
>
Reply | Threaded
Open this post in threaded view
|

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

Valentin Kulichenko
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 <[hidden email]>
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 <[hidden email]>
> wrote:
>
> > Thank you Valentin !!!
> >
> > Regards
> > Saikat
> >
> > On Wed, Oct 26, 2016 at 3:38 AM, Valentin Kulichenko <
> > [hidden email]> wrote:
> >
> >> Hi Saikat,
> >>
> >> I added a comment in the ticket.
> >>
> >> -Val
> >>
> >> On Tue, Oct 25, 2016 at 12:51 PM, Saikat Maitra <
> [hidden email]>
> >> 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
> >> >
> >>
> >
> >
>