Great, thanks! As always, happy to contribute! Thanks, Amir
On Fri, Jun 22, 2018 at 7:32 PM Valentin Kulichenko < [email protected]> wrote: > Amir, > > I merged you change to master and 2.6. Thanks! > > -Val > > On Fri, Jun 22, 2018 at 4:21 PM Amir Akhmedov <[email protected]> > wrote: > >> Val, >> I replied to it already :) >> >> Thanks, >> Amir >> >> >> On Fri, Jun 22, 2018 at 7:20 PM Valentin Kulichenko < >> [email protected]> 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 <[email protected]> >>> 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 < >>>> [email protected]> 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 <[email protected]> >>>>> 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 <[email protected]>: >>>>>> >>>>>> > 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 >>>>>> > >>>>>> >>>>>
