fluent-plugin-numeric-monitor 0.1.1 → 0.1.2

data/fluent-plugin-numeric-monitor.gemspec

@@ -1,7 +1,7 @@
 # -*- encoding: utf-8 -*-
 Gem::Specification.new do |gem|
   gem.name          = "fluent-plugin-numeric-monitor"
-  gem.version       = "0.1.1"
+  gem.version       = "0.1.2"
   gem.authors       = ["TAGOMORI Satoshi"]
   gem.email         = ["tagomoris@gmail.com"]
   gem.description   = %q{Fluentd plugin to calculate min/max/avg/Xpercentile values, and emit these data as message}

data/lib/fluent/plugin/out_numeric_monitor.rb

@@ -195,7 +195,6 @@ class Fluent::NumericMonitorOutput < Fluent::Output
       next if value.nil?
 
       value = value.to_f
-
       if min.nil? or min > value
         min = value
       end
@@ -209,7 +208,7 @@ class Fluent::NumericMonitorOutput < Fluent::Output
         sample.push(value)
       end
     end
-    if sample.size > @samples_limit
+    if @percentiles && sample.size > @samples_limit
       (sample.size - @samples_limit / 2).to_i.times do
         sample.delete_at(rand(sample.size))
       end

data/test/plugin/test_out_numeric_monitor.rb

@@ -64,4 +64,22 @@ class NumericMonitorOutputTest < Test::Unit::TestCase
     assert_equal 8, r1['tag1_percentile_90']
     assert_equal 100, r1['tag1_num']
   end
+
+  def test_without_percentiles
+    d = create_driver(%[
+      unit minute
+      tag testmonitor
+      monitor_key x1
+    ], 'test')
+    d.run do
+      d.emit({'x1' => 1})
+      d.emit({'x1' => 2})
+      d.emit({'x1' => 3})
+    end
+    r = d.instance.flush
+    assert_equal 1, r['test_min']
+    assert_equal 3, r['test_max']
+    assert_equal 2, r['test_avg']
+    assert_equal 3, r['test_num']
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-numeric-monitor
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-08-07 00:00:00.000000000 Z
+date: 2012-12-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fluentd