Add top level stats path validation #49

norberthu · 2015-02-23T12:18:47Z

This change validates stats path and warns if any entries do not
start with carbon|servers|stats.

This change validates stats path and warns if any entries do not start with carbon|servers|stats.

JeremyGrosser · 2015-02-23T19:39:30Z

It'd be nice if you could configure these prefixes, rather than having them hardcoded. Not everybody follows the same schema for stats.

@theatrus

``` - CLOCK_REALTIME_COARSE - clock_getres: 0 seconds | 4000000 nanoseconds Cost (min): 12 cycles Cost (avg): 53 cycles Cost (max): 37061 cycles - GETTIMEOFDAY - gettimeofday_getres: 1us Cost (min): 309 cycles Cost (avg): 429 cycles Cost (max): 107227 cycles ``` /cc @lyft/observability /cc @theatrus

@theatrus

``` - CLOCK_REALTIME_COARSE - clock_getres: 0 seconds | 4000000 nanoseconds Cost (min): 12 cycles Cost (avg): 53 cycles Cost (max): 37061 cycles - GETTIMEOFDAY - gettimeofday_getres: 1us Cost (min): 309 cycles Cost (avg): 429 cycles Cost (max): 107227 cycles ``` /cc @lyft/observability /cc @theatrus

CLAassistant · 2019-06-27T13:09:01Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

@theatrus

``` - CLOCK_REALTIME_COARSE - clock_getres: 0 seconds | 4000000 nanoseconds Cost (min): 12 cycles Cost (avg): 53 cycles Cost (max): 37061 cycles - GETTIMEOFDAY - gettimeofday_getres: 1us Cost (min): 309 cycles Cost (avg): 429 cycles Cost (max): 107227 cycles ``` /cc @lyft/observability /cc @theatrus

Add top level stats path validation

02f60de

This change validates stats path and warns if any entries do not start with carbon|servers|stats.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add top level stats path validation #49

Add top level stats path validation #49

norberthu commented Feb 23, 2015

JeremyGrosser commented Feb 23, 2015

CLAassistant commented Jun 27, 2019 •

edited

Loading

Add top level stats path validation #49

Are you sure you want to change the base?

Add top level stats path validation #49

Conversation

norberthu commented Feb 23, 2015

JeremyGrosser commented Feb 23, 2015

CLAassistant commented Jun 27, 2019 • edited Loading

CLAassistant commented Jun 27, 2019 •

edited

Loading