PR review

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

PR review

Alexey Goncharuk
Folks,

Can I get a review of pull requests 508 [1] and 509 [2]  for Jira tickets
2707 [3] and 2709 [4] respectively? The changes are quite small, TC looks
ok.

Thanks,
AG

[1] https://github.com/apache/ignite/pull/508
[2] https://github.com/apache/ignite/pull/509
[3] https://issues.apache.org/jira/browse/IGNITE-2707
[4] https://issues.apache.org/jira/browse/IGNITE-2709
Reply | Threaded
Open this post in threaded view
|

Re: PR review

Semyon Boikov
Hi Alexey,

I reviewed your changes, added comments in tickets.

Semyon

On Wed, Feb 24, 2016 at 3:47 AM, Alexey Goncharuk <
[hidden email]> wrote:

> Folks,
>
> Can I get a review of pull requests 508 [1] and 509 [2]  for Jira tickets
> 2707 [3] and 2709 [4] respectively? The changes are quite small, TC looks
> ok.
>
> Thanks,
> AG
>
> [1] https://github.com/apache/ignite/pull/508
> [2] https://github.com/apache/ignite/pull/509
> [3] https://issues.apache.org/jira/browse/IGNITE-2707
> [4] https://issues.apache.org/jira/browse/IGNITE-2709
>
Reply | Threaded
Open this post in threaded view
|

Re: PR review

Alexey Goncharuk
Thanks, Semyon!

I've addressed the comments.