IGNITE-4878 ready for review

[michael.griggs]

https://issues.apache.org/jira/browse/IGNITE-4878

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

[dmagda]

Replied in "Distribution of keys to partitions” discussion.

—
Denis

> On Mar 31, 2017, at 3:21 AM, michael.griggs <[hidden email]> 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.

[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!

[Sergi]

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

> 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.
>

[dmagda]

Folks, does it make sense to close the ticket than?

—
Denis

> On Apr 10, 2017, at 12:35 AM, Sergi Vladykin <[hidden email]> wrote:
>
> 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 <[hidden email]>:
>
>> 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.
>>