Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-03-13 Thread ALEKSEY KUZNETSOV
Thanks! пн, 13 мар. 2017 г. в 17:24, Alexey Goncharuk : > Aleksey, > > Thanks, I merged your changes to master. Please resolve the ticket. > > 2017-03-13 12:33 GMT+03:00 ALEKSEY KUZNETSOV : > > > I've fixed all your remarks. Plz, review them > > > > пн, 13 мар. 2017 г. в 11:17, Alexey Goncharuk <

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-03-13 Thread Alexey Goncharuk
Aleksey, Thanks, I merged your changes to master. Please resolve the ticket. 2017-03-13 12:33 GMT+03:00 ALEKSEY KUZNETSOV : > I've fixed all your remarks. Plz, review them > > пн, 13 мар. 2017 г. в 11:17, Alexey Goncharuk >: > > > My apologies, the comments were not committed to GitHub. They ar

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-03-13 Thread ALEKSEY KUZNETSOV
I've fixed all your remarks. Plz, review them пн, 13 мар. 2017 г. в 11:17, Alexey Goncharuk : > My apologies, the comments were not committed to GitHub. They are there > now. > > 2017-03-13 11:13 GMT+03:00 ALEKSEY KUZNETSOV : > > > Hi! This is my PR link : https://github.com/apache/ignite/pull/15

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-03-13 Thread Alexey Goncharuk
My apologies, the comments were not committed to GitHub. They are there now. 2017-03-13 11:13 GMT+03:00 ALEKSEY KUZNETSOV : > Hi! This is my PR link : https://github.com/apache/ignite/pull/1517. You > must have mixed it up, for there is *no* comments to address. > > пт, 10 мар. 2017 г. в 20:10, A

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-03-13 Thread ALEKSEY KUZNETSOV
Hi! This is my PR link : https://github.com/apache/ignite/pull/1517. You must have mixed it up, for there is *no* comments to address. пт, 10 мар. 2017 г. в 20:10, Alexey Goncharuk : > Yes, but I have added a few more today :) > > 2017-03-10 19:03 GMT+03:00 ALEKSEY KUZNETSOV : > > > Hmm, i have f

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-03-10 Thread Alexey Goncharuk
Yes, but I have added a few more today :) 2017-03-10 19:03 GMT+03:00 ALEKSEY KUZNETSOV : > Hmm, i have fixed all your remarks more than 20 days ago. > > пт, 10 мар. 2017 г. в 15:12, Alexey Goncharuk >: > > > Alexey, > > > > Looks good. I've left my comments in the PR, please address them and I >

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-03-10 Thread ALEKSEY KUZNETSOV
Hmm, i have fixed all your remarks more than 20 days ago. пт, 10 мар. 2017 г. в 15:12, Alexey Goncharuk : > Alexey, > > Looks good. I've left my comments in the PR, please address them and I will > merge it. > > 2017-03-10 10:24 GMT+03:00 ALEKSEY KUZNETSOV : > > > plz review ticket again > > > >

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-03-10 Thread Alexey Goncharuk
Alexey, Looks good. I've left my comments in the PR, please address them and I will merge it. 2017-03-10 10:24 GMT+03:00 ALEKSEY KUZNETSOV : > plz review ticket again > > чт, 9 мар. 2017 г. в 10:28, ALEKSEY KUZNETSOV : > > > plz review ticket again > > > > вт, 28 февр. 2017 г. в 14:14, ALEKSEY K

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-03-09 Thread ALEKSEY KUZNETSOV
plz review ticket again чт, 9 мар. 2017 г. в 10:28, ALEKSEY KUZNETSOV : > plz review ticket again > > вт, 28 февр. 2017 г. в 14:14, ALEKSEY KUZNETSOV >: > > plz review ticket again > > пн, 20 февр. 2017 г. в 11:14, Alexey Goncharuk >: > > Thanks, Aleksey, > > I will take a look this week. > > -

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-03-09 Thread ALEKSEY KUZNETSOV
plz review ticket again вт, 28 февр. 2017 г. в 14:14, ALEKSEY KUZNETSOV : > plz review ticket again > > пн, 20 февр. 2017 г. в 11:14, Alexey Goncharuk >: > > Thanks, Aleksey, > > I will take a look this week. > > --AG > > 2017-02-20 10:25 GMT+03:00 ALEKSEY KUZNETSOV : > > > Hi! Review my PR agai

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-02-28 Thread ALEKSEY KUZNETSOV
plz review ticket again пн, 20 февр. 2017 г. в 11:14, Alexey Goncharuk : > Thanks, Aleksey, > > I will take a look this week. > > --AG > > 2017-02-20 10:25 GMT+03:00 ALEKSEY KUZNETSOV : > > > Hi! Review my PR again, plz - https://github.com/apache/ignite/pull/1517 > > > > пт, 17 февр. 2017 г. в 1

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-02-20 Thread Alexey Goncharuk
Thanks, Aleksey, I will take a look this week. --AG 2017-02-20 10:25 GMT+03:00 ALEKSEY KUZNETSOV : > Hi! Review my PR again, plz - https://github.com/apache/ignite/pull/1517 > > пт, 17 февр. 2017 г. в 14:44, ALEKSEY KUZNETSOV >: > > > thanx! my next PR review will be in up source > > > > пт, 1

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-02-19 Thread ALEKSEY KUZNETSOV
Hi! Review my PR again, plz - https://github.com/apache/ignite/pull/1517 пт, 17 февр. 2017 г. в 14:44, ALEKSEY KUZNETSOV : > thanx! my next PR review will be in up source > > пт, 17 февр. 2017 г. в 13:05, Alexey Goncharuk >: > > Aleksey, > > I added a comment on GitHub, however, the community is

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-02-17 Thread ALEKSEY KUZNETSOV
thanx! my next PR review will be in up source пт, 17 февр. 2017 г. в 13:05, Alexey Goncharuk : > Aleksey, > > I added a comment on GitHub, however, the community is moving towards the > UpSource review tool, so I suggest you open a PR review in Ignite UpSource: > http://reviews.ignite.apache.org/

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-02-17 Thread Alexey Goncharuk
Aleksey, I added a comment on GitHub, however, the community is moving towards the UpSource review tool, so I suggest you open a PR review in Ignite UpSource: http://reviews.ignite.apache.org/ignite/ After you've registered, you should be able to open a review. --AG 2017-02-17 11:13 GMT+03:00 A

PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-02-17 Thread ALEKSEY KUZNETSOV
Again, plz, review my PR : https://github.com/apache/ignite/pull/1517 https://issues.apache.org/jira/browse/IGNITE-1178 -- *Best Regards,* *Kuznetsov Aleksey*

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-02-16 Thread ALEKSEY KUZNETSOV
thankx чт, 16 февр. 2017 г. в 11:29, Alexey Kuznetsov : > Hi! > > I did minor code-style review directly on GitHub. Please fix. > > But some one who knows GridCacheProcessor.java should do one more review. > > On Thu, Feb 16, 2017 at 2:38 PM, ALEKSEY KUZNETSOV < > alkuznetsov...@gmail.com > > wro

Re: PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-02-16 Thread Alexey Kuznetsov
Hi! I did minor code-style review directly on GitHub. Please fix. But some one who knows GridCacheProcessor.java should do one more review. On Thu, Feb 16, 2017 at 2:38 PM, ALEKSEY KUZNETSOV wrote: > Plz, review my PR : https://github.com/apache/ignite/pull/1517 > > https://issues.apache.org/j

PR IGNITE-1178 fix for NPE in GridCacheProcessor.onKernalStop()

2017-02-15 Thread ALEKSEY KUZNETSOV
Plz, review my PR : https://github.com/apache/ignite/pull/1517 https://issues.apache.org/jira/browse/IGNITE-1178 -- *Best Regards,* *Kuznetsov Aleksey*