'Implement IgniteSemaphore data structure' Review

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

'Implement IgniteSemaphore data structure' Review

Anton Vinogradov
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
Reply | Threaded
Open this post in threaded view
|

Re: 'Implement IgniteSemaphore data structure' Review

dsetrakyan
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 <[hidden email]>
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
>
Reply | Threaded
Open this post in threaded view
|

Re: 'Implement IgniteSemaphore data structure' Review

Anton Vinogradov
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 <[hidden email]>
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 <
> [hidden email]>
> 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
> >
>