Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[API-537] Add support for the binary metrics #441

Merged
merged 2 commits into from
Aug 6, 2021

Commits on Aug 4, 2021

  1. Add support for the binary metrics

    Implemented the mechanism to compress the metrics we collect in a
    way that is compatible with the server-side.
    
    With this change, the latest Management Center versions are able to
    show statistics regarding the Python client.
    
    For backward compatibility, we also continue sending metrics in
    a comma separated key-value format (client attributes).
    
    In this PR, I also significantly cleaned up the statistics service.
    In the old code, we were trying to reuse the results of some
    calls. In the benchmarks I did, I saw that it didn't really improve
    much. (New approach approx. 0.5 ms, old approach approx. 0.4 ms per
    statistics collection) This really small difference does not justify
    the need for complexity so I went with the simpler approach.
    
    Also, aligned the log levels with the Java client for the various
    warnings.
    
    Also, bumped the version of psutil to 5.8.0.
    mdumandag committed Aug 4, 2021
    Configuration menu
    Copy the full SHA
    3e990f1 View commit details
    Browse the repository at this point in the history

Commits on Aug 5, 2021

  1. address review comments

    mdumandag committed Aug 5, 2021
    Configuration menu
    Copy the full SHA
    ddfc21d View commit details
    Browse the repository at this point in the history