Skip to content

Commit 99e8e3b

Browse files
committed
Renamed Method to ClearDefaultDimensions and removed the non default Dimensions from the clear method
1 parent fdbbd11 commit 99e8e3b

File tree

6 files changed

+11
-12
lines changed

6 files changed

+11
-12
lines changed

libraries/src/AWS.Lambda.Powertools.Metrics/IMetrics.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,5 +100,5 @@ void PushSingleMetric(string metricName, double value, MetricUnit unit, string n
100100
/// <summary>
101101
/// Clears both default dimensions and dimensions lists
102102
/// </summary>
103-
void ClearAllDimensions();
103+
void ClearDefaultDimensions();
104104
}

libraries/src/AWS.Lambda.Powertools.Metrics/Metrics.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -216,9 +216,9 @@ void IMetrics.Flush(bool metricsOverflow)
216216
/// <summary>
217217
/// Clears both default dimensions and dimensions lists
218218
/// </summary>
219-
void IMetrics.ClearAllDimensions()
219+
void IMetrics.ClearDefaultDimensions()
220220
{
221-
_context.ClearAllDimensions();
221+
_context.ClearDefaultDimensions();
222222
}
223223

224224
/// <summary>
@@ -329,9 +329,9 @@ public static void SetDefaultDimensions(Dictionary<string, string> defaultDimens
329329
/// <summary>
330330
/// Clears both default dimensions and dimensions lists
331331
/// </summary>
332-
public static void ClearAllDimensions()
332+
public static void ClearDefaultDimensions()
333333
{
334-
_instance.ClearAllDimensions();
334+
_instance.ClearDefaultDimensions();
335335
}
336336

337337
/// <summary>

libraries/src/AWS.Lambda.Powertools.Metrics/Model/Metadata.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -179,8 +179,8 @@ internal Dictionary<string, string> ExpandAllDimensionSets()
179179
/// <summary>
180180
/// Clears both default dimensions and dimensions lists
181181
/// </summary>
182-
internal void ClearAllDimensions()
182+
internal void ClearDefaultDimensions()
183183
{
184-
_metricDirective.ClearAllDimensions();
184+
_metricDirective.ClearDefaultDimensions();
185185
}
186186
}

libraries/src/AWS.Lambda.Powertools.Metrics/Model/MetricDirective.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -229,9 +229,8 @@ internal Dictionary<string, string> ExpandAllDimensionSets()
229229
/// <summary>
230230
/// Clears both default dimensions and dimensions lists
231231
/// </summary>
232-
internal void ClearAllDimensions()
232+
internal void ClearDefaultDimensions()
233233
{
234-
Dimensions.Clear();
235234
DefaultDimensions.Clear();
236235
}
237236
}

libraries/src/AWS.Lambda.Powertools.Metrics/Model/MetricsContext.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -166,8 +166,8 @@ public string Serialize()
166166
/// <summary>
167167
/// Clears both default dimensions and dimensions lists
168168
/// </summary>
169-
public void ClearAllDimensions()
169+
public void ClearDefaultDimensions()
170170
{
171-
_rootNode.AWS.ClearAllDimensions();
171+
_rootNode.AWS.ClearDefaultDimensions();
172172
}
173173
}

libraries/tests/AWS.Lambda.Powertools.Metrics.Tests/ClearDimensionsTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public void WhenClearAllDimensions_NoDimensionsInOutput()
3535
// Act
3636
handler.OnEntry(eventArgs);
3737

38-
Metrics.ClearAllDimensions();
38+
Metrics.ClearDefaultDimensions();
3939
Metrics.AddMetric($"Metric Name", 1, MetricUnit.Count);
4040

4141
handler.OnExit(eventArgs);

0 commit comments

Comments
 (0)