IGNITE-13069, please review pull request

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

IGNITE-13069, please review pull request

Konstantin Sirotkin
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-13069, please review pull request

dmagda
Nick, Alex,

Could any of you please check the changes by Konstantin? If there are other
committers who can review the pull-request please loop them in.

-
Denis


On Mon, Jul 27, 2020 at 11:37 PM Konstantin Sirotkin <
[hidden email]> wrote:
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-13069, please review pull request

Konstantin Sirotkin
In reply to this post by Konstantin Sirotkin
Now I am working on breaking this PR into smaller parts.

On 2020/07/28 18:55:51, Denis Magda <[hidden email]> wrote:

> Nick, Alex,>
>
> Could any of you please check the changes by Konstantin? If there are other>
> committers who can review the pull-request please loop them in.>
>
> ->
> Denis>
>
>
> On Mon, Jul 27, 2020 at 11:37 PM Konstantin Sirotkin <>
> [hidden email]> wrote:>
>
> > Hello>
> >>
> > Please review pull request - https://github.com/apache/ignite/pull/8063 <>
> > https://github.com/apache/ignite/pull/8063>>
> > Apache jira issue - https://issues.apache.org/jira/browse/IGNITE-13069 <>
> > https://issues.apache.org/jira/browse/IGNITE-13069>>
> >>
> > Thanks>
> >>
> > Konstantin Sirotkin>
>  


---
Kind Regards,
Konstantin Sirotkin

Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-13069, please review pull request

Konstantin Sirotkin
Hello!

Done, I divided the PR into five smaller ones 8090-8093,8095.
Can someone please check the changes now?

Thanks.

---
Kind Regards,
Konstantin Sirotkin

> On 28 Jul 2020, at 22:18, Konstantin Sirotkin <[hidden email]> wrote:
>
> Now I am working on breaking this PR into smaller parts.
>
> On 2020/07/28 18:55:51, Denis Magda <[hidden email] <http://apache.org/>> wrote:
> > Nick, Alex,>
> >
> > Could any of you please check the changes by Konstantin? If there are other>
> > committers who can review the pull-request please loop them in.>
> >
> > ->
> > Denis>
> >
> >
> > On Mon, Jul 27, 2020 at 11:37 PM Konstantin Sirotkin <>
> > [hidden email] <mailto:[hidden email]>> wrote:>
> >
> > > Hello>
> > >>
> > > Please review pull request - https://github.com/apache/ignite/pull/8063 <https://github.com/apache/ignite/pull/8063> <>
> > > https://github.com/apache/ignite/pull/8063 <https://github.com/apache/ignite/pull/8063>>>
> > > Apache jira issue - https://issues.apache.org/jira/browse/IGNITE-13069 <https://issues.apache.org/jira/browse/IGNITE-13069> <>
> > > https://issues.apache.org/jira/browse/IGNITE-13069 <https://issues.apache.org/jira/browse/IGNITE-13069>>>
> > >>
> > > Thanks>
> > >>
> > > Konstantin Sirotkin>
> >  
>
>
> ---
> Kind Regards,
> Konstantin Sirotkin
>

Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-13069, please review pull request

Alexey Goncharuk
Konstantin, thank you, I will take a look.

пт, 31 июл. 2020 г. в 19:04, Konstantin Sirotkin <[hidden email]>:

> Hello!
>
> Done, I divided the PR into five smaller ones 8090-8093,8095.
> Can someone please check the changes now?
>
> Thanks.
>
> ---
> Kind Regards,
> Konstantin Sirotkin
>
> On 28 Jul 2020, at 22:18, Konstantin Sirotkin <[hidden email]>
> wrote:
>
> Now I am working on breaking this PR into smaller parts.
>
> On 2020/07/28 18:55:51, Denis Magda <[hidden email]> wrote:
> > Nick, Alex,>
> >
> > Could any of you please check the changes by Konstantin? If there are
> other>
> > committers who can review the pull-request please loop them in.>
> >
> > ->
> > Denis>
> >
> >
> > On Mon, Jul 27, 2020 at 11:37 PM Konstantin Sirotkin <>
> > [hidden email]> wrote:>
> >
> > > Hello>
> > >>
> > > Please review pull request -
> https://github.com/apache/ignite/pull/8063 <>
> > > https://github.com/apache/ignite/pull/8063>>
> > > Apache jira issue - https://issues.apache.org/jira/browse/IGNITE-13069
> <>
> > > https://issues.apache.org/jira/browse/IGNITE-13069>>
> > >>
> > > Thanks>
> > >>
> > > Konstantin Sirotkin>
> >
>
>
> ---
> Kind Regards,
> Konstantin Sirotkin
>
>
>