feat(host-metrics): update host metrics to collect metrics in batch (o…

…pen-telemetry#1450) Co-authored-by: Haddas Bronfman <[email protected]>
dynatrace-oss-contrib · Apr 6, 2023 · 6c708d1 · 6c708d1
1 parent fd5f87e
commit 6c708d1
Show file tree

Hide file tree

Showing 5 changed files with 141 additions and 268 deletions.
diff --git a/packages/opentelemetry-host-metrics/src/BaseMetrics.ts b/packages/opentelemetry-host-metrics/src/BaseMetrics.ts
@@ -23,8 +23,6 @@ import { VERSION } from './version';
  * Metrics Collector Configuration
  */
 export interface MetricsCollectorConfig {
-  // maximum timeout to wait for stats collection default is 500ms
-  maxTimeoutUpdateMS?: number;
   // Meter Provider
   meterProvider?: metrics.MeterProvider;
   // Character to be used to join metrics - default is "."
@@ -35,22 +33,18 @@ export interface MetricsCollectorConfig {
   url?: string;
 }
 
-export const DEFAULT_MAX_TIMEOUT_UPDATE_MS = 500;
 const DEFAULT_NAME = 'opentelemetry-host-metrics';
 
 /**
  * Base Class for metrics
  */
 export abstract class BaseMetrics {
   protected _logger = api.diag;
-  protected _maxTimeoutUpdateMS: number;
   protected _meter: api.Meter;
   private _name: string;
 
   constructor(config: MetricsCollectorConfig) {
     this._name = config.name || DEFAULT_NAME;
-    this._maxTimeoutUpdateMS =
-      config.maxTimeoutUpdateMS || DEFAULT_MAX_TIMEOUT_UPDATE_MS;
     const meterProvider =
       config.meterProvider! || api.metrics.getMeterProvider();
     if (!config.meterProvider) {