Skip to content

Commit

Permalink
Re-enable aggregation by default (#171)
Browse files Browse the repository at this point in the history
* Re-enable aggregation by default

Setting was enabled by default in #158, but this change was reverted by
accident when #157 was merged.

* Disable aggregation for telemetry tests
  • Loading branch information
vickenty authored Dec 6, 2021
1 parent a54208e commit 812c040
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ String tag() {
public static final boolean DEFAULT_BLOCKING = false;
public static final boolean DEFAULT_ENABLE_TELEMETRY = true;

public static final boolean DEFAULT_ENABLE_AGGREGATION = false;
public static final boolean DEFAULT_ENABLE_AGGREGATION = true;

public static final String CLIENT_TAG = "client:java";
public static final String CLIENT_VERSION_TAG = "client_version:";
Expand Down
1 change: 1 addition & 0 deletions src/test/java/com/timgroup/statsd/TelemetryTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,7 @@ public StatsDNonBlockingTelemetry(final String prefix, final int queueSize, Stri
.processorWorkers(processorWorkers)
.senderWorkers(senderWorkers)
.blocking(blocking)
.enableAggregation(false)
.enableTelemetry(enableTelemetry)
.telemetryFlushInterval(telemetryFlushInterval)
.resolve());
Expand Down

0 comments on commit 812c040

Please sign in to comment.