Re: Contribution merged: IGNITE-3263 (Affinity function must check for null keys)

2016-08-17 Thread Saikat Maitra
Thank you Denis !!! Regards Saikat On Wed, Aug 17, 2016 at 4:55 AM, Denis Magda wrote: > Saikat, > > Thanks a lot for keep making Apache Ignite better. I’ve just merged you > recent improvements > https://issues.apache.org/jira/browse/IGNITE-3263 > > Regards, > Denis > >

Contribution merged: IGNITE-3263 (Affinity function must check for null keys)

2016-08-16 Thread Denis Magda
Saikat, Thanks a lot for keep making Apache Ignite better. I’ve just merged you recent improvements https://issues.apache.org/jira/browse/IGNITE-3263 Regards, Denis

Re: IGNITE-3263 Affinity function must check for null keys

2016-08-09 Thread Saikat Maitra
Thank you Denis. I will update the PR accordingly. Regards Saikat On Tue, Aug 9, 2016 at 3:39 AM, Denis Magda wrote: > Hi Saikat, > > Thanks for keep contributing! I’ve reviewed your changes and left comments > in JIRA. > > — > Denis > > > On Aug 7, 2016, at 2:11 AM,

Re: IGNITE-3263 Affinity function must check for null keys

2016-08-08 Thread Denis Magda
Hi Saikat, Thanks for keep contributing! I’ve reviewed your changes and left comments in JIRA. — Denis > On Aug 7, 2016, at 2:11 AM, Saikat Maitra wrote: > > Hi, > > I have raised PR[1] for the following jira ticket[2]. > > Please review and let me know if any

IGNITE-3263 Affinity function must check for null keys

2016-08-07 Thread Saikat Maitra
Hi, I have raised PR[1] for the following jira ticket[2]. Please review and let me know if any changes required. I have added an example to demonstrate the change is working fine and tested but was not able to reproduce the exception scenario using unit tests. Regards, Saikat [1]

[jira] [Created] (IGNITE-3263) Affinity function must check for null keys

2016-06-06 Thread Denis Magda (JIRA)
Denis Magda created IGNITE-3263: --- Summary: Affinity function must check for null keys Key: IGNITE-3263 URL: https://issues.apache.org/jira/browse/IGNITE-3263 Project: Ignite Issue Type: Bug