Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

Update stats format when read all values #570

Merged
merged 5 commits into from
Jul 1, 2021

Conversation

yixinglu
Copy link
Contributor

Response format when exec $ curl -G 'http://localhost:5922/stats'

original format

value=0
name=num_query_errors.sum.5
value=0
name=num_query_errors.sum.60
value=0
name=num_query_errors.sum.600
...

new format

slow_query_latency_us.avg.5=0
num_slow_queries.rate.5=0
num_slow_queries.rate.60=0
num_slow_queries.sum.600=0
num_slow_queries.sum.3600=0
...

@yixinglu yixinglu added the ready-for-testing PR: ready for the CI test label Jun 24, 2021
Aiee
Aiee previously approved these changes Jun 24, 2021
@yixinglu yixinglu merged commit a1b3787 into vesoft-inc:master Jul 1, 2021
@yixinglu yixinglu deleted the fix-stats-fmt branch July 1, 2021 08:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants