You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by Ken Cheng <kc...@gmail.com> on 2016/01/30 09:10:11 UTC
PR : Please help do the code review for IGNITE-1069
Dear Ignite Developers,
I have raised a PR https://github.com/apache/ignite/pull/440 for bug
https://issues.apache.org/jira/browse/IGNITE-1069
please help do a code review.
Thanks,
kcheng
Re: PR : Please help do the code review for IGNITE-1069
Posted by Ken Cheng <kc...@gmail.com>.
Hi Pavel,
Raised PR. All the tests passed
http://204.14.53.151/viewLog.html?buildId=105755&buildTypeId=IgniteTests_IgniteVisorConsoleTestDetach&tab=buildResultsDiv
Thanks,
kcheng
On Mon, Feb 1, 2016 at 2:46 PM, Ken Cheng <kc...@gmail.com> wrote:
> Hi Pavel,
>
> I will take care of it.
>
>
>
> Thanks,
> kcheng
>
> On Mon, Feb 1, 2016 at 2:42 PM, Pavel Konstantinov <
> pkonstantinov@gridgain.com> wrote:
>
>> Guys,
>>
>> I've added my thoughts in the ticket.
>>
>> Ken, could you please take a look?
>>
>
>
Re: PR : Please help do the code review for IGNITE-1069
Posted by Ken Cheng <kc...@gmail.com>.
Hi Pavel,
I will take care of it.
Thanks,
kcheng
On Mon, Feb 1, 2016 at 2:42 PM, Pavel Konstantinov <
pkonstantinov@gridgain.com> wrote:
> Guys,
>
> I've added my thoughts in the ticket.
>
> Ken, could you please take a look?
>
Re: PR : Please help do the code review for IGNITE-1069
Posted by Pavel Konstantinov <pk...@gridgain.com>.
Guys,
I've added my thoughts in the ticket.
Ken, could you please take a look?
Re: PR : Please help do the code review for IGNITE-1069
Posted by Alexey Kuznetsov <ak...@gridgain.com>.
Hi Ken,
That was my fault. Fixed.
Thanks!
On Mon, Feb 1, 2016 at 10:36 AM, Ken Cheng <kc...@gmail.com> wrote:
> Hi Alexey,
>
> Thank you very much!
>
> but I noticed the PR https://github.com/apache/ignite/pull/440 is still at
> open status
>
> Thanks,
> kcheng
>
> On Mon, Feb 1, 2016 at 11:16 AM, Alexey Kuznetsov <akuznetsov@gridgain.com
> >
> wrote:
>
> > Ken,
> >
> > I done with review, all looks good, I merged your PR to master.
> > Thanks for your contribution.
> >
> >
> > On Sat, Jan 30, 2016 at 11:19 PM, Ken Cheng <kc...@gmail.com>
> wrote:
> >
> > > Hi Alexey Kuznetsov,
> > >
> > > *All the related test cases are passed, please check the detail in
> jira.
> > I
> > > have paste the build link.*
> > >
> > > Thanks,
> > > kcheng
> > >
> > > On Sat, Jan 30, 2016 at 7:03 PM, Alexey Kuznetsov <
> > akuznetsov@gridgain.com
> > > >
> > > wrote:
> > >
> > > > Hi Ken!
> > > > I will review you PR shortly.
> > > >
> > > >
> > > > On Sat, Jan 30, 2016 at 4:03 PM, Denis Magda <dm...@gridgain.com>
> > > wrote:
> > > >
> > > > > Hi Ken,
> > > > >
> > > > > I think that Alex Kuznetsov as a maintainer of visorcmd will be
> able
> > to
> > > > > review your contribution.
> > > > >
> > > > > Regards,
> > > > > Denis
> > > > >
> > > > > > On 30 янв. 2016 г., at 11:59, Ken Cheng <kc...@gmail.com>
> > > wrote:
> > > > > >
> > > > > > @Ilya Suntsov
> > > > > > <
> https://issues.apache.org/jira/secure/ViewProfile.jspa?name=ustas
> > >,
> > > > > >
> > > > > > You logged this issue, please help to reivew it. Thanks.
> > > > > >
> > > > > > Thanks,
> > > > > > kcheng
> > > > > >
> > > > > > On Sat, Jan 30, 2016 at 4:10 PM, Ken Cheng <kcheng.mvp@gmail.com
> >
> > > > wrote:
> > > > > >
> > > > > >>
> > > > > >> Dear Ignite Developers,
> > > > > >>
> > > > > >>
> > > > > >> I have raised a PR https://github.com/apache/ignite/pull/440
> for
> > > bug
> > > > > >> https://issues.apache.org/jira/browse/IGNITE-1069
> > > > > >>
> > > > > >>
> > > > > >> please help do a code review.
> > > > > >>
> > > > > >>
> > > > > >>
> > > > > >> Thanks,
> > > > > >> kcheng
> > > > > >>
> > > > >
> > > > >
> > > >
> > > >
> > > > --
> > > > Alexey Kuznetsov
> > > > GridGain Systems
> > > > www.gridgain.com
> > > >
> > >
> >
> >
> >
> > --
> > Alexey Kuznetsov
> > GridGain Systems
> > www.gridgain.com
> >
>
--
Alexey Kuznetsov
GridGain Systems
www.gridgain.com
Re: PR : Please help do the code review for IGNITE-1069
Posted by Ken Cheng <kc...@gmail.com>.
Hi Alexey,
Thank you very much!
but I noticed the PR https://github.com/apache/ignite/pull/440 is still at
open status
Thanks,
kcheng
On Mon, Feb 1, 2016 at 11:16 AM, Alexey Kuznetsov <ak...@gridgain.com>
wrote:
> Ken,
>
> I done with review, all looks good, I merged your PR to master.
> Thanks for your contribution.
>
>
> On Sat, Jan 30, 2016 at 11:19 PM, Ken Cheng <kc...@gmail.com> wrote:
>
> > Hi Alexey Kuznetsov,
> >
> > *All the related test cases are passed, please check the detail in jira.
> I
> > have paste the build link.*
> >
> > Thanks,
> > kcheng
> >
> > On Sat, Jan 30, 2016 at 7:03 PM, Alexey Kuznetsov <
> akuznetsov@gridgain.com
> > >
> > wrote:
> >
> > > Hi Ken!
> > > I will review you PR shortly.
> > >
> > >
> > > On Sat, Jan 30, 2016 at 4:03 PM, Denis Magda <dm...@gridgain.com>
> > wrote:
> > >
> > > > Hi Ken,
> > > >
> > > > I think that Alex Kuznetsov as a maintainer of visorcmd will be able
> to
> > > > review your contribution.
> > > >
> > > > Regards,
> > > > Denis
> > > >
> > > > > On 30 янв. 2016 г., at 11:59, Ken Cheng <kc...@gmail.com>
> > wrote:
> > > > >
> > > > > @Ilya Suntsov
> > > > > <https://issues.apache.org/jira/secure/ViewProfile.jspa?name=ustas
> >,
> > > > >
> > > > > You logged this issue, please help to reivew it. Thanks.
> > > > >
> > > > > Thanks,
> > > > > kcheng
> > > > >
> > > > > On Sat, Jan 30, 2016 at 4:10 PM, Ken Cheng <kc...@gmail.com>
> > > wrote:
> > > > >
> > > > >>
> > > > >> Dear Ignite Developers,
> > > > >>
> > > > >>
> > > > >> I have raised a PR https://github.com/apache/ignite/pull/440 for
> > bug
> > > > >> https://issues.apache.org/jira/browse/IGNITE-1069
> > > > >>
> > > > >>
> > > > >> please help do a code review.
> > > > >>
> > > > >>
> > > > >>
> > > > >> Thanks,
> > > > >> kcheng
> > > > >>
> > > >
> > > >
> > >
> > >
> > > --
> > > Alexey Kuznetsov
> > > GridGain Systems
> > > www.gridgain.com
> > >
> >
>
>
>
> --
> Alexey Kuznetsov
> GridGain Systems
> www.gridgain.com
>
Re: PR : Please help do the code review for IGNITE-1069
Posted by Alexey Kuznetsov <ak...@gridgain.com>.
Ken,
I done with review, all looks good, I merged your PR to master.
Thanks for your contribution.
On Sat, Jan 30, 2016 at 11:19 PM, Ken Cheng <kc...@gmail.com> wrote:
> Hi Alexey Kuznetsov,
>
> *All the related test cases are passed, please check the detail in jira. I
> have paste the build link.*
>
> Thanks,
> kcheng
>
> On Sat, Jan 30, 2016 at 7:03 PM, Alexey Kuznetsov <akuznetsov@gridgain.com
> >
> wrote:
>
> > Hi Ken!
> > I will review you PR shortly.
> >
> >
> > On Sat, Jan 30, 2016 at 4:03 PM, Denis Magda <dm...@gridgain.com>
> wrote:
> >
> > > Hi Ken,
> > >
> > > I think that Alex Kuznetsov as a maintainer of visorcmd will be able to
> > > review your contribution.
> > >
> > > Regards,
> > > Denis
> > >
> > > > On 30 янв. 2016 г., at 11:59, Ken Cheng <kc...@gmail.com>
> wrote:
> > > >
> > > > @Ilya Suntsov
> > > > <https://issues.apache.org/jira/secure/ViewProfile.jspa?name=ustas>,
> > > >
> > > > You logged this issue, please help to reivew it. Thanks.
> > > >
> > > > Thanks,
> > > > kcheng
> > > >
> > > > On Sat, Jan 30, 2016 at 4:10 PM, Ken Cheng <kc...@gmail.com>
> > wrote:
> > > >
> > > >>
> > > >> Dear Ignite Developers,
> > > >>
> > > >>
> > > >> I have raised a PR https://github.com/apache/ignite/pull/440 for
> bug
> > > >> https://issues.apache.org/jira/browse/IGNITE-1069
> > > >>
> > > >>
> > > >> please help do a code review.
> > > >>
> > > >>
> > > >>
> > > >> Thanks,
> > > >> kcheng
> > > >>
> > >
> > >
> >
> >
> > --
> > Alexey Kuznetsov
> > GridGain Systems
> > www.gridgain.com
> >
>
--
Alexey Kuznetsov
GridGain Systems
www.gridgain.com
Re: PR : Please help do the code review for IGNITE-1069
Posted by Ken Cheng <kc...@gmail.com>.
Hi Alexey Kuznetsov,
*All the related test cases are passed, please check the detail in jira. I
have paste the build link.*
Thanks,
kcheng
On Sat, Jan 30, 2016 at 7:03 PM, Alexey Kuznetsov <ak...@gridgain.com>
wrote:
> Hi Ken!
> I will review you PR shortly.
>
>
> On Sat, Jan 30, 2016 at 4:03 PM, Denis Magda <dm...@gridgain.com> wrote:
>
> > Hi Ken,
> >
> > I think that Alex Kuznetsov as a maintainer of visorcmd will be able to
> > review your contribution.
> >
> > Regards,
> > Denis
> >
> > > On 30 янв. 2016 г., at 11:59, Ken Cheng <kc...@gmail.com> wrote:
> > >
> > > @Ilya Suntsov
> > > <https://issues.apache.org/jira/secure/ViewProfile.jspa?name=ustas>,
> > >
> > > You logged this issue, please help to reivew it. Thanks.
> > >
> > > Thanks,
> > > kcheng
> > >
> > > On Sat, Jan 30, 2016 at 4:10 PM, Ken Cheng <kc...@gmail.com>
> wrote:
> > >
> > >>
> > >> Dear Ignite Developers,
> > >>
> > >>
> > >> I have raised a PR https://github.com/apache/ignite/pull/440 for bug
> > >> https://issues.apache.org/jira/browse/IGNITE-1069
> > >>
> > >>
> > >> please help do a code review.
> > >>
> > >>
> > >>
> > >> Thanks,
> > >> kcheng
> > >>
> >
> >
>
>
> --
> Alexey Kuznetsov
> GridGain Systems
> www.gridgain.com
>
Re: PR : Please help do the code review for IGNITE-1069
Posted by Alexey Kuznetsov <ak...@gridgain.com>.
Hi Ken!
I will review you PR shortly.
On Sat, Jan 30, 2016 at 4:03 PM, Denis Magda <dm...@gridgain.com> wrote:
> Hi Ken,
>
> I think that Alex Kuznetsov as a maintainer of visorcmd will be able to
> review your contribution.
>
> Regards,
> Denis
>
> > On 30 янв. 2016 г., at 11:59, Ken Cheng <kc...@gmail.com> wrote:
> >
> > @Ilya Suntsov
> > <https://issues.apache.org/jira/secure/ViewProfile.jspa?name=ustas>,
> >
> > You logged this issue, please help to reivew it. Thanks.
> >
> > Thanks,
> > kcheng
> >
> > On Sat, Jan 30, 2016 at 4:10 PM, Ken Cheng <kc...@gmail.com> wrote:
> >
> >>
> >> Dear Ignite Developers,
> >>
> >>
> >> I have raised a PR https://github.com/apache/ignite/pull/440 for bug
> >> https://issues.apache.org/jira/browse/IGNITE-1069
> >>
> >>
> >> please help do a code review.
> >>
> >>
> >>
> >> Thanks,
> >> kcheng
> >>
>
>
--
Alexey Kuznetsov
GridGain Systems
www.gridgain.com
Re: PR : Please help do the code review for IGNITE-1069
Posted by Denis Magda <dm...@gridgain.com>.
Hi Ken,
I think that Alex Kuznetsov as a maintainer of visorcmd will be able to review your contribution.
Regards,
Denis
> On 30 янв. 2016 г., at 11:59, Ken Cheng <kc...@gmail.com> wrote:
>
> @Ilya Suntsov
> <https://issues.apache.org/jira/secure/ViewProfile.jspa?name=ustas>,
>
> You logged this issue, please help to reivew it. Thanks.
>
> Thanks,
> kcheng
>
> On Sat, Jan 30, 2016 at 4:10 PM, Ken Cheng <kc...@gmail.com> wrote:
>
>>
>> Dear Ignite Developers,
>>
>>
>> I have raised a PR https://github.com/apache/ignite/pull/440 for bug
>> https://issues.apache.org/jira/browse/IGNITE-1069
>>
>>
>> please help do a code review.
>>
>>
>>
>> Thanks,
>> kcheng
>>
Re: PR : Please help do the code review for IGNITE-1069
Posted by Ken Cheng <kc...@gmail.com>.
@Ilya Suntsov
<https://issues.apache.org/jira/secure/ViewProfile.jspa?name=ustas>,
You logged this issue, please help to reivew it. Thanks.
Thanks,
kcheng
On Sat, Jan 30, 2016 at 4:10 PM, Ken Cheng <kc...@gmail.com> wrote:
>
> Dear Ignite Developers,
>
>
> I have raised a PR https://github.com/apache/ignite/pull/440 for bug
> https://issues.apache.org/jira/browse/IGNITE-1069
>
>
> please help do a code review.
>
>
>
> Thanks,
> kcheng
>