Skip to content

Commit

Permalink
Add missing management.metrics.export.signalfx properties
Browse files Browse the repository at this point in the history
  • Loading branch information
wilkinsona committed Jul 21, 2023
1 parent ae59172 commit cb5c506
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright 2012-2019 the original author or authors.
* Copyright 2012-2023 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -54,6 +54,11 @@ public class SignalFxProperties extends StepRegistryProperties {
*/
private String source;

/**
* Whether to emit cumulative histogram buckets.
*/
private boolean pushCumulativeHistogram = false;

@Override
public Duration getStep() {
return this.step;
Expand Down Expand Up @@ -88,4 +93,12 @@ public void setSource(String source) {
this.source = source;
}

public boolean isPushCumulativeHistogram() {
return this.pushCumulativeHistogram;
}

public void setPushCumulativeHistogram(boolean pushCumulativeHistogram) {
this.pushCumulativeHistogram = pushCumulativeHistogram;
}

}
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright 2012-2020 the original author or authors.
* Copyright 2012-2023 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -54,4 +54,9 @@ public String source() {
return get(SignalFxProperties::getSource, SignalFxConfig.super::source);
}

@Override
public boolean publishCumulativeHistogram() {
return get(SignalFxProperties::isPushCumulativeHistogram, SignalFxConfig.super::publishCumulativeHistogram);
}

}

0 comments on commit cb5c506

Please sign in to comment.