Merge lp:~sidnei/txstatsd/carbonize into lp:txstatsd

Proposed by Sidnei da Silva
Status: Merged
Approved by: Sidnei da Silva
Approved revision: 63
Merged at revision: 58
Proposed branch: lp:~sidnei/txstatsd/carbonize
Merge into: lp:txstatsd
Diff against target: 0 lines
To merge this branch: bzr merge lp:~sidnei/txstatsd/carbonize
Reviewer Review Type Date Requested Status
Curtis Caravone (community) Approve
Lucio Torre (community) Approve
Review via email: mp+84175@code.launchpad.net

Commit message

Replace GraphiteClientProtocol with CarbonClientManager

Description of the change

Replaces our naive GraphiteClientProtocol implementation by carbon's own
CarbonClientManager. As a bonus we get to support ConsistentHashing out of the
box. Configuration options for routing and XXX fixes to come later.

To post a comment you must log in.
Revision history for this message
Lucio Torre (lucio.torre) :
review: Approve
Revision history for this message
Curtis Caravone (caravone) wrote :

Deleting code and replacing it with a library component that's maintained by somebody else? I love it!

review: Approve
Revision history for this message
Ubuntu One Server Tarmac Bot (ubuntuone-server-tarmac) wrote :
Download full text (20.6 KiB)

The attempt to merge lp:~sidnei/txstatsd/carbonize into lp:txstatsd failed. Below is the output from the failed tests.

txstatsd.tests.metrics.test_distinct
  TestDistinct
    test_all ... [OK]
  TestDistinctMetricReporter
    test_reports ... [OK]
  TestHash
    test_chi_square ... [SKIPPED]
    test_hash_chars ... [OK]
  TestPlugin
    test_factory ... [OK]
  TestZeros
    test_zeros ... [OK]
txstatsd.tests.metrics.test_histogrammetric
  TestHistogramReporterMetric
    test_histogram_of_numbers_1_through_10000 ... [OK]
    test_histogram_with_zero_recorded_values ... [OK]
txstatsd.tests.metrics.test_timermetric
  TestBlankTimerMetric
    test_count ... [OK]
    test_fifteen_minute_rate ... [OK]
    test_five_minute_rate ... [OK]
    test_max ... [OK]
    test_mean ... [OK]
    test_mean_rate ... [OK]
    test_min ... [OK]
    test_no_values ... [OK]
    test_one_minute_rate ... [OK]
    test_percentiles ... [OK]
    test_std_dev ... [OK]
  TestTimingSeriesEvents
    test_count ... [OK]
    test_max ... [OK]
    test_mean ... [OK]
    test_min ... [OK]
    test_percentiles ... [OK]
    test_std_dev ... [OK]
    test_values ... [OK]
txstatsd.tests.stats.test_ewma
  TestEwmaFifteenMinute
    test_eight_minutes ... [OK]
    test_eleven_minutes ... [OK]
    test_fifteen_minutes ... [OK]
    test_first_tick ... [OK]
    test_five_minutes ... [OK]
    test_four_minutes ... [OK]
    test_fourteen_minutes ... [OK]
    test_nine_minutes ... [OK]
    test_one_minute ... ...

Revision history for this message
Ubuntu One Server Tarmac Bot (ubuntuone-server-tarmac) wrote :

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

Preview Diff

Empty

Subscribers

People subscribed via source and target branches