You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@ignite.apache.org by Anton Vinogradov <av...@gridgain.com> on 2015/10/02 10:19:32 UTC

'Implement IgniteSemaphore data structure' Review

Val,

Vladisav prepared pull request related to Implement IgniteSemaphore data
structure <https://issues.apache.org/jira/browse/IGNITE-638>.
Pull request seems to be ready for review.

Could you please review it?
PR url: https://github.com/apache/ignite/pull/120

Re: 'Implement IgniteSemaphore data structure' Review

Posted by Anton Vinogradov <av...@gridgain.com>.

Vladisav,
Please move issue to the Patch Available state and mention pull request
(start a title of the pull request from 'IGNITE-638').

Val,
Do you have opportunity to review pull request?

On Fri, Oct 2, 2015 at 11:47 AM, Dmitriy Setrakyan <ds...@apache.org>
wrote:

> Any reason the pull request is not mentioned in the ticket? Also, the
> ticket should be moved to the Patch Available state, right?
>
> D.
>
> On Fri, Oct 2, 2015 at 10:19 AM, Anton Vinogradov <
> avinogradov@gridgain.com>
> wrote:
>
> > Val,
> >
> > Vladisav prepared pull request related to Implement IgniteSemaphore data
> > structure <https://issues.apache.org/jira/browse/IGNITE-638>.
> > Pull request seems to be ready for review.
> >
> > Could you please review it?
> > PR url: https://github.com/apache/ignite/pull/120
> >
>

Re: 'Implement IgniteSemaphore data structure' Review

Posted by Dmitriy Setrakyan <ds...@apache.org>.

Any reason the pull request is not mentioned in the ticket? Also, the
ticket should be moved to the Patch Available state, right?

D.

On Fri, Oct 2, 2015 at 10:19 AM, Anton Vinogradov <av...@gridgain.com>
wrote:

> Val,
>
> Vladisav prepared pull request related to Implement IgniteSemaphore data
> structure <https://issues.apache.org/jira/browse/IGNITE-638>.
> Pull request seems to be ready for review.
>
> Could you please review it?
> PR url: https://github.com/apache/ignite/pull/120
>