Skip to content

Commit 5baaa91

Browse files
committed
Defaults moved to the interface
1 parent ad9a91c commit 5baaa91

File tree

3 files changed

+7
-12
lines changed

3 files changed

+7
-12
lines changed

profiler/src/main/java/com/splunk/opentelemetry/profiler/snapshot/SnapshotProfilingConfiguration.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,10 @@
2929
public interface SnapshotProfilingConfiguration {
3030
double MAX_SELECTION_PROBABILITY = 1.0;
3131
double DEFAULT_SELECTION_PROBABILITY = 0.01;
32+
int DEFAULT_STACK_DEPTH = 1024;
33+
long DEFAULT_SAMPLING_INTERVAL = 10;
34+
long DEFAULT_EXPORT_INTERVAL = 5000;
35+
int DEFAULT_STAGING_CAPACITY = 2000;
3236

3337
void log();
3438

profiler/src/main/java/com/splunk/opentelemetry/profiler/snapshot/SnapshotProfilingDeclarativeConfiguration.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,6 @@ public class SnapshotProfilingDeclarativeConfiguration implements SnapshotProfil
2727
Logger.getLogger(SnapshotProfilingDeclarativeConfiguration.class.getName());
2828
private final DeclarativeConfigProperties profilingConfig;
2929

30-
private static final int DEFAULT_STACK_DEPTH = 1024;
31-
private static final long DEFAULT_SAMPLING_INTERVAL = 10;
32-
private static final long DEFAULT_EXPORT_INTERVAL = 5000;
33-
private static final int DEFAULT_STAGING_CAPACITY = 2000;
34-
3530
private static final String ROOT_NODE_NAME = "callgraphs";
3631

3732
SnapshotProfilingDeclarativeConfiguration(DeclarativeConfigProperties profilingConfig) {

profiler/src/main/java/com/splunk/opentelemetry/profiler/snapshot/SnapshotProfilingEnvVarsConfiguration.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,6 @@ class SnapshotProfilingEnvVarsConfiguration implements SnapshotProfilingConfigur
3434
static final String EXPORT_INTERVAL_KEY = "splunk.snapshot.profiler.export.interval";
3535
static final String STAGING_CAPACITY_KEY = "splunk.snapshot.profiler.staging.capacity";
3636

37-
private static final int DEFAULT_STACK_DEPTH = 1024;
38-
private static final Duration DEFAULT_SAMPLING_INTERVAL = Duration.ofMillis(10);
39-
private static final Duration DEFAULT_EXPORT_INTERVAL = Duration.ofSeconds(5);
40-
private static final int DEFAULT_STAGING_CAPACITY = 2000;
41-
4237
SnapshotProfilingEnvVarsConfiguration(ConfigProperties properties) {
4338
this.properties = properties;
4439
}
@@ -89,12 +84,13 @@ public int getStackDepth() {
8984

9085
@Override
9186
public Duration getSamplingInterval() {
92-
return properties.getDuration(SAMPLING_INTERVAL_KEY, DEFAULT_SAMPLING_INTERVAL);
87+
return properties.getDuration(
88+
SAMPLING_INTERVAL_KEY, Duration.ofMillis(DEFAULT_SAMPLING_INTERVAL));
9389
}
9490

9591
@Override
9692
public Duration getExportInterval() {
97-
return properties.getDuration(EXPORT_INTERVAL_KEY, DEFAULT_EXPORT_INTERVAL);
93+
return properties.getDuration(EXPORT_INTERVAL_KEY, Duration.ofMillis(DEFAULT_EXPORT_INTERVAL));
9894
}
9995

10096
@Override

0 commit comments

Comments
 (0)