----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17204/#review32661 -----------------------------------------------------------
Ship it! Ship It! - Will Vaughan On Jan. 23, 2014, 5:41 p.m., Matthew Hayes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17204/ > ----------------------------------------------------------- > > (Updated Jan. 23, 2014, 5:41 p.m.) > > > Review request for DataFu. > > > Repository: datafu > > > Description > ------- > > Now the reservoir is cleared before each use. > > > Diffs > ----- > > src/java/datafu/pig/sampling/ReservoirSample.java 747b28c > src/java/datafu/pig/sampling/WeightedReservoirSample.java 0792411 > test/pig/datafu/test/pig/sampling/SamplingTests.java 0dbbda0 > test/pig/datafu/test/pig/sampling/WeightedReservoirSamplingTests.java > f507cbd > > Diff: https://reviews.apache.org/r/17204/diff/ > > > Testing > ------- > > Added unit tests that reproduced the issue, which now pass. > > > Thanks, > > Matthew Hayes > >
