Merge lp://staging/~theiw/txstatsd/txstatsd-coda-hale-counters into lp://staging/txstatsd
Proposed by
Ian Wilkinson
| Status: | Merged |
|---|---|
| Approved by: | Sidnei da Silva |
| Approved revision: | 34 |
| Merged at revision: | 34 |
| Proposed branch: | lp://staging/~theiw/txstatsd/txstatsd-coda-hale-counters |
| Merge into: | lp://staging/txstatsd |
| Diff against target: |
284 lines (+156/-20) 7 files modified
txstatsd/metrics/countermetric.py (+76/-0) txstatsd/metrics/extendedmetrics.py (+39/-0) txstatsd/metrics/metermetric.py (+3/-1) txstatsd/server/configurableprocessor.py (+26/-8) txstatsd/server/processor.py (+4/-0) txstatsd/tests/test_processor.py (+7/-10) txstatsd/version.py (+1/-1) |
| To merge this branch: | bzr merge lp://staging/~theiw/txstatsd/txstatsd-coda-hale-counters |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Sidnei da Silva | Approve | ||
| Lucio Torre (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Include support for reporting counter metrics, fixes an issue with the prefix for meter metrics.
Description of the change
Include support for reporting counter metrics.
The specialised ExtendedMetrics was introducing imagining an interface away from that of StatsD.
(This also fixes an issue with the prefix for meter metrics.)
To post a comment you must log in.
Revision history for this message
| Lucio Torre (lucio.torre) | # |
review:
Approve
Revision history for this message
| Sidnei da Silva (sidnei) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
