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