Aleksey, Thanks for your contribution! I've merged this PR into master branch. See JIRA issue comment for details.
On Fri, Mar 10, 2017 at 2:58 PM, Andrey Gura <ag...@apache.org> wrote: > Aleksey, > > I don't see any new changes. So I'll check TC and merge changes today. > > On Fri, Mar 10, 2017 at 10:20 AM, ALEKSEY KUZNETSOV > <alkuznetsov...@gmail.com> wrote: >> Hi! Can u plz review ticket once more >> >> вт, 7 мар. 2017 г. в 18:52, Andrey Gura <ag...@apache.org>: >> >>> Aleksey, thanks a lot! >>> >>> Answered in JIRA ticket. >>> >>> On Tue, Mar 7, 2017 at 1:27 PM, ALEKSEY KUZNETSOV >>> <alkuznetsov...@gmail.com> wrote: >>> > Hi! I have fixed all sources. Plz, review it again >>> > >>> > пн, 6 мар. 2017 г. в 15:43, Andrey Gura <ag...@apache.org>: >>> > >>> >> Aleksey, thanks! >>> >> >>> >> I answered in JIRA ticket. >>> >> >>> >> On Mon, Mar 6, 2017 at 10:56 AM, ALEKSEY KUZNETSOV >>> >> <alkuznetsov...@gmail.com> wrote: >>> >> > I've fixed the comments. >>> >> > http://reviews.ignite.apache.org/ignite/review/IGNT-CR-98 >>> >> > >>> >> > пт, 3 мар. 2017 г. в 19:23, Andrey Gura <ag...@apache.org>: >>> >> > >>> >> >> Aleksey, >>> >> >> >>> >> >> GitHub isn't official review tool in Apache Ignite community. There >>> >> >> are two ways for code review: upsource and comments in JIRA tickets. >>> >> >> So, I think, we should finish review of this ticket in upsource. >>> >> >> >>> >> >> On Thu, Mar 2, 2017 at 11:30 AM, ALEKSEY KUZNETSOV >>> >> >> <alkuznetsov...@gmail.com> wrote: >>> >> >> > lets review code at github rather than upsource later on. Because, >>> the >>> >> >> > later is too slow and bring no substantial benefits compared github >>> >> >> > >>> >> >> > ср, 1 мар. 2017 г. в 18:04, Andrey Gura <ag...@apache.org>: >>> >> >> > >>> >> >> >> Hi, Aleksey! >>> >> >> >> >>> >> >> >> Thank you for contribution! >>> >> >> >> >>> >> >> >> I've reviewed your changes and have some comments (mostly >>> cosmetic). >>> >> >> >> Could you please fix this comment? See review in Upsource for >>> >> details. >>> >> >> >> >>> >> >> >> On Tue, Feb 28, 2017 at 2:17 PM, ALEKSEY KUZNETSOV >>> >> >> >> <alkuznetsov...@gmail.com> wrote: >>> >> >> >> > Plz, review my PR : >>> >> >> >> > http://reviews.ignite.apache.org/ignite/review/IGNT-CR-98 >>> >> >> >> > or https://github.com/apache/ignite/pull/1545 >>> >> >> >> > -- >>> >> >> >> > >>> >> >> >> > *Best Regards,* >>> >> >> >> > >>> >> >> >> > *Kuznetsov Aleksey* >>> >> >> >> >>> >> >> > -- >>> >> >> > >>> >> >> > *Best Regards,* >>> >> >> > >>> >> >> > *Kuznetsov Aleksey* >>> >> >> >>> >> > -- >>> >> > >>> >> > *Best Regards,* >>> >> > >>> >> > *Kuznetsov Aleksey* >>> >> >>> > -- >>> > >>> > *Best Regards,* >>> > >>> > *Kuznetsov Aleksey* >>> >> -- >> >> *Best Regards,* >> >> *Kuznetsov Aleksey*