Amir, I merged you change to master and 2.6. Thanks!
-Val On Fri, Jun 22, 2018 at 4:21 PM Amir Akhmedov <amir.akhme...@gmail.com> wrote: > Val, > I replied to it already :) > > Thanks, > Amir > > > On Fri, Jun 22, 2018 at 7:20 PM Valentin Kulichenko < > valentin.kuliche...@gmail.com> wrote: > >> Amir, >> >> Thanks for quick reaction. I added a follow up question in the ticket. >> >> -Val >> >> On Fri, Jun 22, 2018 at 3:48 PM Amir Akhmedov <amir.akhme...@gmail.com> >> wrote: >> >>> Hi Val, >>> Thanks for your comments. I replied in the ticket with my vision of the >>> issue and how I tried to solve it. Please check it and let me know. >>> >>> Thanks, >>> Amir >>> >>> >>> On Fri, Jun 22, 2018 at 5:42 PM Valentin Kulichenko < >>> valentin.kuliche...@gmail.com> wrote: >>> >>>> Hi Amir, >>>> >>>> I reviewed the changes and I'm not sure I understood how they fix they >>>> issue. I left more detailed comment in the ticket, can you please clarify? >>>> >>>> -Val >>>> >>>> On Fri, Jun 22, 2018 at 9:53 AM Dmitry Pavlov <dpavlov....@gmail.com> >>>> wrote: >>>> >>>>> Hi Amir, >>>>> >>>>> let me say sincere thank you for continuing to contribute. >>>>> >>>>> Bumping up this thread. >>>>> >>>>> Igniters, who has an expertise here? >>>>> >>>>> >>>>> вс, 17 июн. 2018 г. в 17:59, Amir Akhmedov <amir.akhme...@gmail.com>: >>>>> >>>>> > Hi All, >>>>> > Can you please review my changes for IGNITE-8740. >>>>> > >>>>> > PR: https://github.com/apache/ignite/pull/4208 >>>>> > TC: >>>>> > >>>>> > >>>>> https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll >>>>> > >>>>> > Thanks, >>>>> > Amir >>>>> > >>>>> >>>>
