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 > >> > >> >
