Commit 7a500026 authored by Marc Celani's avatar Marc Celani Committed by Facebook Github Bot

Move a function to .cpp file

Summary: Some links are complaining of multiple definitions. Move to the cpp file to resolve.

Reviewed By: yfeldblum

Differential Revision: D7806874

fbshipit-source-id: f25342d647505b676ba3659400c63d6a59344024
parent 410c0a02
......@@ -25,19 +25,10 @@ namespace detail {
QuantileEstimates estimatesFromDigest(
const TDigest& digest,
Range<const double*> quantiles) {
QuantileEstimates result;
result.quantiles.reserve(quantiles.size());
result.sum = digest.sum();
result.count = digest.count();
for (auto it = quantiles.begin(); it != quantiles.end(); ++it) {
result.quantiles.push_back(
std::make_pair(*it, digest.estimateQuantile(*it)));
}
return result;
}
Range<const double*> quantiles);
} // namespace detail
template <typename ClockT>
SimpleQuantileEstimator<ClockT>::SimpleQuantileEstimator()
: bufferedDigest_(std::chrono::seconds{1}, 1000, 100) {}
......
......@@ -17,6 +17,23 @@
#include <folly/stats/QuantileEstimator-defs.h>
namespace folly {
namespace detail {
QuantileEstimates estimatesFromDigest(
const TDigest& digest,
Range<const double*> quantiles) {
QuantileEstimates result;
result.quantiles.reserve(quantiles.size());
result.sum = digest.sum();
result.count = digest.count();
for (auto it = quantiles.begin(); it != quantiles.end(); ++it) {
result.quantiles.push_back(
std::make_pair(*it, digest.estimateQuantile(*it)));
}
return result;
}
} // namespace detail
template class SimpleQuantileEstimator<std::chrono::steady_clock>;
template class SlidingWindowQuantileEstimator<std::chrono::steady_clock>;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment