fix: default effect counters should be right type. (#2373)

Co-authored-by: Tim <[email protected]>
Effect-TS · Mar 20, 2024 · 9bbde5b · 9bbde5b
1 parent 64a250d
commit 9bbde5b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/.changeset/small-pets-fold.md b/.changeset/small-pets-fold.md
@@ -0,0 +1,5 @@
+---
+"effect": patch
+---
+
+Use incremental counters instead of up-down for runtime metrics
diff --git a/packages/effect/src/internal/fiberRuntime.ts b/packages/effect/src/internal/fiberRuntime.ts
@@ -71,13 +71,13 @@ import * as tracer from "./tracer.js"
 import * as version from "./version.js"
 
 /** @internal */
-export const fiberStarted = metric.counter("effect_fiber_started")
+export const fiberStarted = metric.counter("effect_fiber_started", { incremental: true })
 /** @internal */
 export const fiberActive = metric.counter("effect_fiber_active")
 /** @internal */
-export const fiberSuccesses = metric.counter("effect_fiber_successes")
+export const fiberSuccesses = metric.counter("effect_fiber_successes", { incremental: true })
 /** @internal */
-export const fiberFailures = metric.counter("effect_fiber_failures")
+export const fiberFailures = metric.counter("effect_fiber_failures", { incremental: true })
 /** @internal */
 export const fiberLifetimes = metric.tagged(
   metric.histogram(