Skip to content

Commit fc266e5

Browse files
authored
[Profiling] Switch to 19Hz sampling frequency (#117757)
* [Profiling] Switch to 19Hz sampling frequency * Fix internalClusterTest
1 parent af9a57e commit fc266e5

File tree

5 files changed

+9
-9
lines changed

5 files changed

+9
-9
lines changed

x-pack/plugin/profiling/src/internalClusterTest/java/org/elasticsearch/xpack/profiling/action/GetStackTracesActionIT.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@ public void testGetStackTracesUnfiltered() throws Exception {
4646
assertEquals(18, stackTrace.fileIds.length);
4747
assertEquals(18, stackTrace.frameIds.length);
4848
assertEquals(18, stackTrace.typeIds.length);
49-
assertEquals(0.0000048475146d, stackTrace.annualCO2Tons, 0.0000000001d);
50-
assertEquals(0.18834d, stackTrace.annualCostsUSD, 0.00001d);
49+
assertEquals(0.0000051026469d, stackTrace.annualCO2Tons, 0.0000000001d);
50+
assertEquals(0.19825d, stackTrace.annualCostsUSD, 0.00001d);
5151
// not determined by default
5252
assertNull(stackTrace.subGroups);
5353

@@ -91,8 +91,8 @@ public void testGetStackTracesGroupedByServiceName() throws Exception {
9191
assertEquals(18, stackTrace.fileIds.length);
9292
assertEquals(18, stackTrace.frameIds.length);
9393
assertEquals(18, stackTrace.typeIds.length);
94-
assertEquals(0.0000048475146d, stackTrace.annualCO2Tons, 0.0000000001d);
95-
assertEquals(0.18834d, stackTrace.annualCostsUSD, 0.00001d);
94+
assertEquals(0.0000051026469d, stackTrace.annualCO2Tons, 0.0000000001d);
95+
assertEquals(0.19825d, stackTrace.annualCostsUSD, 0.00001d);
9696
assertEquals(Long.valueOf(2L), stackTrace.subGroups.getCount("basket"));
9797

9898
assertNotNull(response.getStackFrames());

x-pack/plugin/profiling/src/main/java/org/elasticsearch/xpack/profiling/action/CO2Calculator.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
import java.util.Map;
1313

1414
final class CO2Calculator {
15-
private static final double DEFAULT_SAMPLING_FREQUENCY = 20.0d;
15+
private static final double DEFAULT_SAMPLING_FREQUENCY = 19.0d;
1616
private static final double DEFAULT_CO2_TONS_PER_KWH = 0.000379069d; // unit: metric tons / kWh
1717
private static final double DEFAULT_KILOWATTS_PER_CORE_X86 = 7.0d / 1000.0d; // unit: watt / core
1818
private static final double DEFAULT_KILOWATTS_PER_CORE_ARM64 = 2.8d / 1000.0d; // unit: watt / core

x-pack/plugin/profiling/src/main/java/org/elasticsearch/xpack/profiling/action/CostCalculator.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
import java.util.Map;
1111

1212
final class CostCalculator {
13-
private static final double DEFAULT_SAMPLING_FREQUENCY = 20.0d;
13+
private static final double DEFAULT_SAMPLING_FREQUENCY = 19.0d;
1414
private static final double SECONDS_PER_HOUR = 60 * 60;
1515
private static final double SECONDS_PER_YEAR = SECONDS_PER_HOUR * 24 * 365.0d; // unit: seconds
1616
public static final double DEFAULT_COST_USD_PER_CORE_HOUR = 0.0425d; // unit: USD / (core * hour)

x-pack/plugin/profiling/src/test/java/org/elasticsearch/xpack/profiling/action/CO2CalculatorTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ public void testCreateFromRegularSource() {
7373

7474
double samplingDurationInSeconds = 1_800.0d; // 30 minutes
7575
long samples = 100_000L; // 100k samples
76-
double annualCoreHours = CostCalculator.annualCoreHours(samplingDurationInSeconds, samples, 20.0d);
76+
double annualCoreHours = CostCalculator.annualCoreHours(samplingDurationInSeconds, samples, 19.0d);
7777
CO2Calculator co2Calculator = new CO2Calculator(hostsTable, samplingDurationInSeconds, null, null, null, null);
7878

7979
checkCO2Calculation(co2Calculator.getAnnualCO2Tons(HOST_ID_A, samples), annualCoreHours, 1.135d, 0.0002786d, 7.0d);
@@ -110,7 +110,7 @@ public void testCreateFromMalformedSource() {
110110

111111
double samplingDurationInSeconds = 1_800.0d; // 30 minutes
112112
long samples = 100_000L; // 100k samples
113-
double annualCoreHours = CostCalculator.annualCoreHours(samplingDurationInSeconds, samples, 20.0d);
113+
double annualCoreHours = CostCalculator.annualCoreHours(samplingDurationInSeconds, samples, 19.0d);
114114
CO2Calculator co2Calculator = new CO2Calculator(hostsTable, samplingDurationInSeconds, null, null, null, null);
115115

116116
checkCO2Calculation(co2Calculator.getAnnualCO2Tons(HOST_ID_A, samples), annualCoreHours, 1.135d, 0.0002786d, 7.0d);

x-pack/plugin/profiling/src/test/java/org/elasticsearch/xpack/profiling/action/CostCalculatorTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ public void testCreateFromRegularSource() {
6363

6464
double samplingDurationInSeconds = 1_800.0d; // 30 minutes
6565
long samples = 100_000L; // 100k samples
66-
double annualCoreHours = CostCalculator.annualCoreHours(samplingDurationInSeconds, samples, 20.0d);
66+
double annualCoreHours = CostCalculator.annualCoreHours(samplingDurationInSeconds, samples, 19.0d);
6767
CostCalculator costCalculator = new CostCalculator(hostsTable, samplingDurationInSeconds, null, null, null);
6868

6969
// Checks whether the cost calculation is based on the lookup data.

0 commit comments

Comments
 (0)