Add some base x264 metrics.

[nageru] / metrics.h

diff --git a/metrics.h b/metrics.h
index 6c72ad53ed71c6bc66b8923a643697d0dea16701..2a19e259cc32bf4df11c1eb779d6339b16382fd9 100644 (file)
--- a/metrics.h
+++ b/metrics.h
@@ -9,18 +9,62 @@
 #include <atomic>
 #include <mutex>
 #include <string>
-#include <unordered_map>
+#include <map>
+#include <vector>
 
 class Metrics {
 public:
-       void register_int_metric(const std::string &name, std::atomic<int64_t> *location);
-       void register_double_metric(const std::string &name, std::atomic<double> *location);
+       enum Type {
+               TYPE_COUNTER,
+               TYPE_GAUGE,
+       };
+
+       void add(const std::string &name, std::atomic<int64_t> *location, Type type = TYPE_COUNTER)
+       {
+               add(name, {}, location, type);
+       }
+
+       void add(const std::string &name, std::atomic<double> *location, Type type = TYPE_COUNTER)
+       {
+               add(name, {}, location, type);
+       }
+
+       void add(const std::string &name, const std::vector<std::pair<std::string, std::string>> &labels, std::atomic<int64_t> *location, Type type = TYPE_COUNTER);
+       void add(const std::string &name, const std::vector<std::pair<std::string, std::string>> &labels, std::atomic<double> *location, Type type = TYPE_COUNTER);
+
+       // Only integer histogram, ie. keys are 0..(N-1).
+       void add_histogram(const std::string &name, const std::vector<std::pair<std::string, std::string>> &labels, std::atomic<int64_t> *location, size_t num_elements);
+
        std::string serialize() const;
 
 private:
+       enum DataType {
+               DATA_TYPE_INT64,
+               DATA_TYPE_DOUBLE,
+       };
+
+       struct Metric {
+               DataType data_type;
+               std::string name;
+               std::vector<std::pair<std::string, std::string>> labels;
+               union {
+                       std::atomic<int64_t> *location_int64;
+                       std::atomic<double> *location_double;
+               };
+       };
+
+       // TODO: This needs to be more general.
+       struct Histogram {
+               std::string name;
+               std::vector<std::pair<std::string, std::string>> labels;
+               std::atomic<int64_t> *location_int64;  // First bucket.
+               size_t num_elements;
+       };
+
        mutable std::mutex mu;
-       std::unordered_map<std::string, std::atomic<int64_t> *> int_metrics;
-       std::unordered_map<std::string, std::atomic<double> *> double_metrics;
+       std::map<std::string, Type> types;
+       std::vector<Metric> metrics;
+       std::vector<Histogram> histograms;
 };
 
 extern Metrics global_metrics;