You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by "michael.griggs" <mi...@gridgain.com> on 2017/04/07 17:00:08 UTC

Re: IGNITE-4878 ready for review

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

Posted by Denis Magda <dm...@apache.org>.
Folks, does it make sense to close the ticket than?

—
Denis

> On Apr 10, 2017, at 12:35 AM, Sergi Vladykin <se...@gmail.com> 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 <mi...@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

Posted by Sergi Vladykin <se...@gmail.com>.
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 <mi...@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.
>