Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Exact Aggregator #198

Merged
merged 23 commits into from
Jul 30, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
0f378be
Create exact_aggregator.h
Brandon-Kimberly Jul 21, 2020
16fc838
Create exact_aggregator_test.cc
Brandon-Kimberly Jul 21, 2020
068bc82
Add exact aggregator test
Brandon-Kimberly Jul 21, 2020
28abe7f
Add exact aggregator test
Brandon-Kimberly Jul 21, 2020
2f22b33
Delete exact_aggregator.h temporarily due to name typo
Brandon-Kimberly Jul 23, 2020
8a18583
Create exact_aggregator.h
Brandon-Kimberly Jul 23, 2020
5a56a78
Update enums used to be in line with Aggregator updates
Brandon-Kimberly Jul 24, 2020
a5bd9ee
Update to support Aggregator base class changes
Brandon-Kimberly Jul 24, 2020
1ed40ac
Change name quantile() to get_quantiles()
Brandon-Kimberly Jul 24, 2020
926a97c
Add copy constructor
Brandon-Kimberly Jul 26, 2020
bc4dcb9
Add get_quant_estimation() function
Brandon-Kimberly Jul 26, 2020
3fc6fd1
Add check to __EXCEPTIONS env var before throwing
Brandon-Kimberly Jul 27, 2020
4051e2b
Merge branch 'master' into sketch-agg
Brandon-Kimberly Jul 28, 2020
483b6a9
Fix exact aggregator test name
Brandon-Kimberly Jul 28, 2020
1b78895
Merge checkpoints as well as values
Brandon-Kimberly Jul 28, 2020
291bd69
Format
Brandon-Kimberly Jul 29, 2020
e504ba7
Merge branch 'master' into sketch-agg
Brandon-Kimberly Jul 30, 2020
e7acbf8
Allocate aggs on the stack
Brandon-Kimberly Jul 30, 2020
b571d09
Format
Brandon-Kimberly Jul 30, 2020
c19b607
Remove include
Brandon-Kimberly Jul 30, 2020
234813d
Add exception checks
Brandon-Kimberly Jul 30, 2020
34cfb90
Merge branch 'master' into sketch-agg
Brandon-Kimberly Jul 30, 2020
8f5db42
Format
Brandon-Kimberly Jul 30, 2020
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
164 changes: 164 additions & 0 deletions sdk/include/opentelemetry/sdk/metrics/aggregator/exact_aggregator.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,164 @@
#pragma once

#include "opentelemetry/metrics/instrument.h"
#include "opentelemetry/sdk/metrics/aggregator/aggregator.h"
#include "opentelemetry/version.h"

#include <cmath>
#include <memory>
#include <mutex>
#include <vector>

namespace metrics_api = opentelemetry::metrics;

OPENTELEMETRY_BEGIN_NAMESPACE
namespace sdk
{
namespace metrics
{
/**
* This aggregator has two modes. In-order and quantile estimation.
*
* The first mode simply stores all values sent to the Update()
* function in a vector and maintains the order they were sent in.
*
* The second mode also stores all values sent to the Update()
* function in a vector but sorts this vector when Checkpoint()
* is called. This mode also includes a function, Quantile(),
* that estimates the quantiles of the recorded data.
*
* @tparam T the type of values stored in this aggregator.
*/
template <class T>
class ExactAggregator : public Aggregator<T>
{
public:
ExactAggregator(metrics_api::InstrumentKind kind, bool quant_estimation = false)
{
static_assert(std::is_arithmetic<T>::value, "Not an arithmetic type");
this->kind_ = kind;
this->checkpoint_ = this->values_;
this->agg_kind_ = AggregatorKind::Exact;
quant_estimation_ = quant_estimation;
}

~ExactAggregator() = default;

ExactAggregator(const ExactAggregator &cp)
{
this->values_ = cp.values_;
this->checkpoint_ = cp.checkpoint_;
this->kind_ = cp.kind_;
this->agg_kind_ = cp.agg_kind_;
quant_estimation_ = cp.quant_estimation_;
// use default initialized mutex as they cannot be copied
}

/**
* Receives a captured value from the instrument and adds it to the values_ vector.
*
* @param val, the raw value used in aggregation
*/
void update(T val) override
{
this->mu_.lock();
this->values_.push_back(val);
this->mu_.unlock();
}

/**
* Checkpoints the current values. This function will overwrite the current checkpoint with the
* current value. Sorts the values_ vector if quant_estimation_ == true
*
*/
void checkpoint() override
{
this->mu_.lock();
if (quant_estimation_)
{
std::sort(this->values_.begin(), this->values_.end());
}
this->checkpoint_ = this->values_;
this->values_.clear();
this->mu_.unlock();
}

/**
* Merges two exact aggregators' values_ vectors together.
*
* @param other the aggregator to merge with this aggregator
*/
void merge(const ExactAggregator &other)
{
if (this->kind_ == other.kind_)
{
this->mu_.lock();
// First merge values
this->values_.insert(this->values_.end(), other.values_.begin(), other.values_.end());
// Now merge checkpoints
this->checkpoint_.insert(this->checkpoint_.end(), other.checkpoint_.begin(),
other.checkpoint_.end());
this->mu_.unlock();
}
else
{
// Log error
return;
}
}

/**
* Performs quantile estimation on the checkpoint vector in this aggregator.
* This function only works if quant_estimation_ == true.
* @param q the quantile to estimate. 0 <= q <= 1
* @return the nearest value in the vector to the exact quantile.
*/
T get_quantiles(double q) override
{
if (!quant_estimation_)
{
// Log error
#if __EXCEPTIONS
throw std::domain_error("Exact aggregator is not in quantile estimation mode!");
#else
std::terminate();
#endif
}
if (this->checkpoint_.size() == 0 || q < 0 || q > 1)
{
// Log error
#if __EXCEPTIONS
throw std::invalid_argument("Arg 'q' must be between 0 and 1, inclusive");
#else
std::terminate();
#endif
}
else if (q == 0 || this->checkpoint_.size() == 1)
{
return this->checkpoint_[0];
}
else if (q == 1)
{
return this->checkpoint_[this->checkpoint_.size() - 1];
}
else
{
float position = float(this->checkpoint_.size() - 1) * q;
int ceiling = ceil(position);
return this->checkpoint_[ceiling];
}
}

//////////////////////////ACCESSOR FUNCTIONS//////////////////////////
std::vector<T> get_checkpoint() override { return this->checkpoint_; }

std::vector<T> get_values() override { return this->values_; }

bool get_quant_estimation() override { return quant_estimation_; }

private:
bool quant_estimation_; // Used to switch between in-order and quantile estimation modes
};
} // namespace metrics
} // namespace sdk
OPENTELEMETRY_END_NAMESPACE
11 changes: 11 additions & 0 deletions sdk/test/metrics/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,17 @@ cc_test(
],
)

cc_test(
name = "exact_aggregator_test",
srcs = [
"exact_aggregator_test.cc",
],
deps = [
"//sdk/src/metrics",
"@com_google_googletest//:gtest_main",
],
)

cc_test(
name = "histogram_aggregator_test",
srcs = [
Expand Down
2 changes: 1 addition & 1 deletion sdk/test/metrics/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
foreach(testname meter_provider_sdk_test gauge_aggregator_test
min_max_sum_count_aggregator_test)
min_max_sum_count_aggregator_test exact_aggregator_test)
add_executable(${testname} "${testname}.cc")
target_link_libraries(${testname} ${GTEST_BOTH_LIBRARIES}
${CMAKE_THREAD_LIBS_INIT} opentelemetry_metrics)
Expand Down
Loading