Review Request

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

Review Request

Atri Sharma-2
Hi,

Please help in reviewing:

https://github.com/apache/ignite/pull/8820

Atri
Reply | Threaded
Open this post in threaded view
|

Re: Review Request

dmagda
@Valentin Kulichenko <[hidden email]>, @Nikolay Izhikov
<[hidden email]>, @[hidden email] <[hidden email]>,

I saw you reviewing the ticket. Could you please double-check the changes?
"IGNITE-2399: Implement acquireAndExecute In IgniteSemaphore"


Atri, please put a ticket number and name in the title of an email, so that
community member can see right away what change is being contributed.

-
Denis


On Wed, Mar 3, 2021 at 7:27 AM Atri Sharma <[hidden email]> wrote:

> Hi,
>
> Please help in reviewing:
>
> https://github.com/apache/ignite/pull/8820
>
> Atri
>
Reply | Threaded
Open this post in threaded view
|

Re: Review Request

Valentin Kulichenko
Atri,

I've added my comments in the PR.

-Val

On Wed, Mar 3, 2021 at 7:29 AM Denis Magda <[hidden email]> wrote:

> @Valentin Kulichenko <[hidden email]>, @Nikolay Izhikov
> <[hidden email]>, @[hidden email] <[hidden email]>,
>
> I saw you reviewing the ticket. Could you please double-check the changes?
> "IGNITE-2399: Implement acquireAndExecute In IgniteSemaphore"
>
>
> Atri, please put a ticket number and name in the title of an email, so that
> community member can see right away what change is being contributed.
>
> -
> Denis
>
>
> On Wed, Mar 3, 2021 at 7:27 AM Atri Sharma <[hidden email]> wrote:
>
> > Hi,
> >
> > Please help in reviewing:
> >
> > https://github.com/apache/ignite/pull/8820
> >
> > Atri
> >
>