Bump to 0.8.4

This release fixes a problem with the prometheus sink's take on summaries. In particular, we were destroying the wrong side of the window, meaning all new values were (eventually) being dumped. This bug caught thanks to @dparton. * 23688f6 :: Correct prometheus window summary truncation (#352) Signed-off-by: Brian L. Troutwine <blt@postmates.com>
postmates · Dec 4, 2017 · d91c1fa · d91c1fa
1 parent 23688f6
commit d91c1fa
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -7,7 +7,7 @@ license = "MIT"
 name = "cernan"
 readme = "README.md"
 repository = "https://github.com/postmates/cernan"
-version = "0.8.4-pre"
+version = "0.8.4"
 
 [[bin]]
 name = "cernan"

diff --git a/src/sink/prometheus.rs b/src/sink/prometheus.rs
@@ -138,7 +138,7 @@ impl Accumulator {
                         samples.insert(idx, telem);
                         if samples.len() > cap {
                             let top_idx = samples.len() - cap;
-                            samples.drain(0 .. top_idx);
+                            samples.drain(0..top_idx);
                         }
                     }
                 }