Thank you all for confirming this! 🙂 Does anyone tried to enable it on geode native side? We tried, but without success. Do we need to change anything else except enable-time-statistics to true as this not working?
BR, Mario ________________________________ Šalje: Charlie Black <cbl...@pivotal.io> Poslano: 10. siječnja 2020. 23:00 Prima: dev@geode.apache.org <dev@geode.apache.org> Predmet: Re: enable-time-statistics David, I would say remove the caveat. Thanks for offering. Charlie On Fri, Jan 10, 2020 at 1:55 PM Dave Barnes <dbar...@pivotal.io> wrote: > Sounds like the caveat could be dropped from the user guide. If we have > consensus on that (am I understanding correctly?), I'll initiate a JIRA > ticket. > > On Fri, Jan 10, 2020 at 1:47 PM Jacob Barrett <jbarr...@pivotal.io> wrote: > > > The biggest impact was in recording all the additional stats in the old > > blocking stats implementation. As of 9.8 the stats internals are mostly > > non-blocking. Enabling time stats has very little of any impact now. > > > > > On Jan 10, 2020, at 12:45 PM, Dan Smith <dsm...@pivotal.io> wrote: > > > > > > I personally wouldn't be too worried about enabling time based > > statistics > > > in production. I think we segregated the time statistics because they > do > > > have to call System.nanoTime to measure the elapsed time. At one point > in > > > the history with old JDKs they called System.currentTimeMillis, which > was > > > really expensive. But now I'm not sure the nanoTime calls really have > > that > > > much of an impact compared to the rest of the processing time. > > > > > > -Dan > > > > > > > > >> On Fri, Jan 10, 2020 at 11:25 AM Mario Kevo <mario.k...@est.tech> > > wrote: > > >> > > >> Hi geode-dev, > > >> > > >> We have executed some traffic against Geode servers with time-based > > >> statistics enabled and disabled and we didn't see any performance > > >> difference. > > >> The documentation says: > > >> > > >> > > >> If you need time-based statistics, enable that. Time-based statistics > > >> require statistics sampling and archival. Example: > > >> > > >> statistic-sampling-enabled=true > > >> statistic-archive-file=myStatisticsArchiveFile.gfs > > >> enable-time-statistics=true > > >> > > >> > > >> Note: Time-based statistics can impact system performance and is not > > >> recommended for production environments. > > >> > > >> > > >> Do you know on which part this note referring to? > > >> > > >> > > >> Also we tried to enable time statistics on geode native but without > > >> success. > > >> > > >> We change in geode.properties file this parameter to true but didn't > get > > >> any additional statistics in statistics archive file. > > >> > > >> Do we need also to change something else to enable it or this is not > > >> working for geode-native? > > >> > > >> > > >> BR, > > >> > > >> Mario > > >> > > >> > > > -- Charlie Black | cbl...@pivotal.io
