Re: About operating system statistics

[Alberto Bustamante Reyes <al...@est.tech>]

pull request available: https://github.com/apache/geode/pull/3574

[https://avatars3.githubusercontent.com/u/47359?s=400&v=4]<https://github.com/apache/geode/pull/3574>

GEODE-6660: Remove non-Linux OS statistics classes by alb3rtobr · Pull Request #3574 · apache/geode<https://github.com/apache/geode/pull/3574>
github.com
Thank you for submitting a contribution to Apache Geode. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: For all changes: Is there...



________________________________
De: Alberto Bustamante Reyes <al...@est.tech>
Enviado: martes, 16 de abril de 2019 15:21:34
Para: dev@geode.apache.org
Asunto: RE: About operating system statistics

Thanks for your answer Darrel. I have created a jira to delete these classes.

BR/

Alberto
________________________________
De: Darrel Schneider <ds...@pivotal.io>
Enviado: lunes, 15 de abril de 2019 17:12
Para: dev@geode.apache.org
Asunto: Re: About operating system statistics

The code you have found predates the open source geode project. That code
requires a native, platform dependent, jar to be distributed. We decided
that geode would be pure java. The code that collects Linux stats is pure
java. So I think the code you have found for Solaris, Windows, and OSX has
been "abandoned" and can be deleted.


On Mon, Apr 15, 2019 at 6:54 AM Alberto Bustamante Reyes
<al...@est.tech> wrote:

> Hi all,
>
> I have read in the documentation that the operating system statistics are
> only available for Linux systems, but I have found in the code the
> correspondent classes for OSX, Solaris & Windows stats (package
> org.apache.geode.internal.statistics.platform).
>
> Are these classes being used? Should they be included in the documentation
> or deleted? (the OSX ones are almost empty, so Im not sure if someone is
> working on it or its an 'abandonded feature')
>
> Thanks in advance,
>
> BR/
>
> Alberto
>