Merge lp://staging/~lucio.torre/txstatsd/make-ewma-sampling-faster into lp://staging/txstatsd
Proposed by
Lucio Torre
| Status: | Rejected |
|---|---|
| Rejected by: | Lucio Torre |
| Proposed branch: | lp://staging/~lucio.torre/txstatsd/make-ewma-sampling-faster |
| Merge into: | lp://staging/txstatsd |
| Diff against target: |
125 lines (+36/-10) 2 files modified
txstatsd/stats/exponentiallydecayingsample.py (+12/-8) txstatsd/tests/stats/test_exponentiallydecayingsample.py (+24/-2) |
| To merge this branch: | bzr merge lp://staging/~lucio.torre/txstatsd/make-ewma-sampling-faster |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| txStatsD Developers | Pending | ||
|
Review via email:
|
|||
Description of the change
make ewma sampling take 23% of the time it takes now
To post a comment you must log in.
Revision history for this message
| Lucio Torre (lucio.torre) wrote | # |
Unmerged revisions
- 66. By Lucio Torre
-
make comment consistent
- 65. By Lucio Torre
-
skip long test
- 64. By Lucio Torre
-
fix rescaling
- 63. By Lucio Torre
-
moved to insort
- 62. By Sidnei da Silva
-
[r=lucio.torre] Split prefix from instance name, make ReportingService use instance name as prefix for reporting own metrics
- 61. By Sidnei da Silva
-
- Fix some docstrings
- 60. By Sidnei da Silva
-
- Delay reactor import so a custom one can be installed.
- 59. By Lucio Torre
-
[r=lucio.
torre,sidnei] add routing - 58. By Sidnei da Silva
-
[r=caravone,
lucio.torre] Replace GraphiteClientP rotocol with CarbonClientManager - 57. By Sidnei da Silva
-
- Fix flushing paused message counter [trivial]
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
now it also changes the rescale threshold so the difference is never bigger than 600 because:
In [21]: math.exp(717)
-------
OverflowError Traceback (most recent call last)
/home/lucio/
OverflowError: math range error