Skip to content
This repository has been archived by the owner on Aug 23, 2023. It is now read-only.

Add scaleToSeconds function #970

Merged
merged 2 commits into from
Aug 7, 2018
Merged

Conversation

stivenbb
Copy link
Contributor

@stivenbb stivenbb commented Aug 1, 2018

Native implementation of scaleToSeconds() Graphite function. (http://graphite.readthedocs.io/en/latest/functions.html#graphite.render.functions.scaleToSeconds)

@shanson7
Copy link
Collaborator

shanson7 commented Aug 1, 2018

See #969 about the generated files. Once that's merged, it will clean up this PR a bit.

@stivenbb
Copy link
Contributor Author

stivenbb commented Aug 6, 2018

Speed improvement:

---------- Native scaleToSeconds.json Latencies ----------
Mean: 53.41152ms
50th percentile: 49.204842ms
95th percentile: 78.921391ms
99th percentile: 125.351173ms
Max: 221.644353ms
Success: 100%
Errors: []

---------- Graphite (Python) scaleToSeconds.json Latencies ----------
Mean: 121.113698ms
50th percentile: 112.906348ms
95th percentile: 192.191345ms
99th percentile: 237.401042ms
Max: 395.471465ms

Success: 100%
Errors: []

---------- Speed Improvement ----------
Mean: x2.3
50th percentile: x2.3
95th percentile: x2.4
99th percentile: x1.9
Max: x1.8

@Dieterbe Dieterbe merged commit 2cb286b into grafana:master Aug 7, 2018
@shanson7 shanson7 deleted the scaleToSeconds branch August 21, 2018 20:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants