Re: IGNITE-4878 ready for review

2017-04-10 Thread Sergi Vladykin
Michael,

This code is already changed in my branch for lazy SQL, so I will not merge
your PR.
Thanks for pointing out the issue anyways!

Sergi

2017-04-07 20:00 GMT+03:00 michael.griggs <michael.gri...@gridgain.com>:

> This change is https://issues.apache.org/jira/browse/IGNITE-4878
>
> IgniteH2Indexing can throw java.util.ConcurrentModificationException
>
> Please can you review this (one-line) change.  Thanks!
>
>
>
> --
> View this message in context: http://apache-ignite-
> developers.2346864.n4.nabble.com/IGNITE-4878-ready-for-
> review-tp16007p16284.html
> Sent from the Apache Ignite Developers mailing list archive at Nabble.com.
>


Re: IGNITE-4878 ready for review

2017-04-07 Thread michael.griggs
This change is https://issues.apache.org/jira/browse/IGNITE-4878

IgniteH2Indexing can throw java.util.ConcurrentModificationException

Please can you review this (one-line) change.  Thanks!



--
View this message in context: 
http://apache-ignite-developers.2346864.n4.nabble.com/IGNITE-4878-ready-for-review-tp16007p16284.html
Sent from the Apache Ignite Developers mailing list archive at Nabble.com.


Re: IGNITE-4878 ready for review

2017-03-31 Thread Denis Magda
Replied in "Distribution of keys to partitions” discussion.

—
Denis

> On Mar 31, 2017, at 3:21 AM, michael.griggs <michael.gri...@gridgain.com> 
> wrote:
> 
> https://issues.apache.org/jira/browse/IGNITE-4878
> 
> https://github.com/apache/ignite/pull/1692
> 
> 
> 
> --
> View this message in context: 
> http://apache-ignite-developers.2346864.n4.nabble.com/IGNITE-4878-ready-for-review-tp16007.html
> Sent from the Apache Ignite Developers mailing list archive at Nabble.com.



IGNITE-4878 ready for review

2017-03-31 Thread michael.griggs
https://issues.apache.org/jira/browse/IGNITE-4878

https://github.com/apache/ignite/pull/1692



--
View this message in context: 
http://apache-ignite-developers.2346864.n4.nabble.com/IGNITE-4878-ready-for-review-tp16007.html
Sent from the Apache Ignite Developers mailing list archive at Nabble.com.