Bug/fix sample uniform #23

Merged
merged 2 commits into from May 29, 2012

Conversation

Projects
None yet
2 participants
@paulperegud
Contributor

paulperegud commented May 28, 2012

Following test:

N = 10000,
folsom_metrics:new_histogram(huge),
[ folsom_metrics:notify(huge, X) || X <- lists:seq(1, N) ],
L = folsom_metrics:get_histogram(huge),
Result = proplists:get_value(median, L).

produced result is very close to max value of the test. This is true for N > 1028.

This patch adds test and fixes the error.

joewilliams added a commit that referenced this pull request May 29, 2012

@joewilliams joewilliams merged commit 16e0924 into boundary:master May 29, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment