Review IGNITE-7276 .NET: Enable/disable cache statistics in runtime

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

Review IGNITE-7276 .NET: Enable/disable cache statistics in runtime

Sergey Stronchinskiy
Hello, Igniters

I've been out for some time and wanted to stretch a little bit my
contributor muscles with some newbie tasks. There some changes both to Java
and .NET, can somebody take a quick look please.

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

Best regards.
Reply | Threaded
Open this post in threaded view
|

Re: Review IGNITE-7276 .NET: Enable/disable cache statistics in runtime

Pavel Tupitsyn
Hello Guru,

I'll have a look today, thank you!

On Sat, Feb 15, 2020 at 11:33 PM Guru Stron <[hidden email]>
wrote:

> Hello, Igniters
>
> I've been out for some time and wanted to stretch a little bit my
> contributor muscles with some newbie tasks. There some changes both to Java
> and .NET, can somebody take a quick look please.
>
> https://github.com/apache/ignite/pull/7434
>
> Best regards.
>
Reply | Threaded
Open this post in threaded view
|

Re: Review IGNITE-7276 .NET: Enable/disable cache statistics in runtime

Sergey Stronchinskiy
Hi Pavel,

Thank you for the review!

Can you please also look at
https://issues.apache.org/jira/browse/IGNITE-9017?

On Mon, 17 Feb 2020 at 11:13, Pavel Tupitsyn <[hidden email]> wrote:

> Hello Guru,
>
> I'll have a look today, thank you!
>
> On Sat, Feb 15, 2020 at 11:33 PM Guru Stron <[hidden email]>
> wrote:
>
> > Hello, Igniters
> >
> > I've been out for some time and wanted to stretch a little bit my
> > contributor muscles with some newbie tasks. There some changes both to
> Java
> > and .NET, can somebody take a quick look please.
> >
> > https://github.com/apache/ignite/pull/7434
> >
> > Best regards.
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Review IGNITE-7276 .NET: Enable/disable cache statistics in runtime

Pavel Tupitsyn
Sure, done!

On Thu, Mar 19, 2020 at 12:52 AM Guru Stron <[hidden email]>
wrote:

> Hi Pavel,
>
> Thank you for the review!
>
> Can you please also look at
> https://issues.apache.org/jira/browse/IGNITE-9017?
>
> On Mon, 17 Feb 2020 at 11:13, Pavel Tupitsyn <[hidden email]> wrote:
>
> > Hello Guru,
> >
> > I'll have a look today, thank you!
> >
> > On Sat, Feb 15, 2020 at 11:33 PM Guru Stron <[hidden email]>
> > wrote:
> >
> > > Hello, Igniters
> > >
> > > I've been out for some time and wanted to stretch a little bit my
> > > contributor muscles with some newbie tasks. There some changes both to
> > Java
> > > and .NET, can somebody take a quick look please.
> > >
> > > https://github.com/apache/ignite/pull/7434
> > >
> > > Best regards.
> > >
> >
>