1. 12 Feb, 2016 1 commit
  2. 11 Feb, 2016 3 commits
  3. 09 Feb, 2016 3 commits
  4. 05 Feb, 2016 5 commits
  5. 04 Feb, 2016 3 commits
  6. 03 Feb, 2016 3 commits
  7. 02 Feb, 2016 2 commits
  8. 01 Feb, 2016 5 commits
    • Merge pull request #802 from herder/patch-1 · 199c740c
      Spencer Gibb authored
      Fix typo: zule-core -> zuul-core.
    • Fix broken zuul tests. · 63896485
      Spencer Gibb authored
    • Merge pull request #800 from brharrington/spectator-issue-264 · 3611edd1
      Spencer Gibb authored
      * spectator-issue-264:
        fix registration bug in SpectatorMetricServices
    • fix registration bug in SpectatorMetricServices · 63b0c8c6
      Brian Harrington authored
      The primary change is to fix a memory leak reported in
      Netflix/spectator#264. Each time a gauge was updated it
      was creating a new registration and because the map holds
      a strong reference these would never get collected.
      Further, the aggregate value created by the multiple
      registrations was not correct.
      
      In addition I added some test cases around the various
      inputs and checked that the results were reflected as
      expected in the registry. I noticed the timer values
      had a unit of milliseconds, but it isn't immediately
      clear if the reported value can ever less than 1.0. The
      conversion to long is now delayed until after converting
      to nanoseconds so duration values less than 1.0 will now
      work instead of just recording 0.
      
      For the histogram I changed to just using a cast to `long`
      to avoid boxing to a `Double`. As an FYI for the future,
      there is a DoubleDistributionSummary we have experimented
      with in spectator-ext-sandbox that might be more appropriate
      for this use-case.
  9. 31 Jan, 2016 2 commits
  10. 29 Jan, 2016 2 commits
  11. 28 Jan, 2016 2 commits
  12. 27 Jan, 2016 4 commits
  13. 26 Jan, 2016 3 commits
  14. 25 Jan, 2016 2 commits