Review comments addressed

Author: onurkybsi

instrumentation/failsafe-3.0/library/src/main/java/io/opentelemetry/instrumentation/failsafe/v3_0/FailsafeTelemetry.java

@@ -21,8 +21,7 @@
 import io.opentelemetry.api.metrics.LongCounter;
 import io.opentelemetry.api.metrics.LongHistogram;
 import io.opentelemetry.api.metrics.Meter;
-import java.util.stream.Collectors;
-import java.util.stream.LongStream;
+import java.util.Arrays;
 
 /** Entrypoint for instrumenting Failsafe components. */
 public final class FailsafeTelemetry {
@@ -102,10 +101,7 @@ public <R> RetryPolicy<R> createRetryPolicy(RetryPolicy<R> delegate, String retr
             .histogramBuilder("failsafe.retry_policy.attempts")
             .setDescription("Histogram of number of attempts for each execution.")
             .ofLongs()
-            .setExplicitBucketBoundariesAdvice(
-                LongStream.range(1, userConfig.getMaxAttempts() + 1)
-                    .boxed()
-                    .collect(Collectors.toList()))
+            .setExplicitBucketBoundariesAdvice(Arrays.asList(1L, 2L, 3L, 5L))
             .build();
     Attributes attributes = Attributes.of(RETRY_POLICY_NAME, retryPolicyName);
     return RetryPolicy.builder(userConfig)

instrumentation/failsafe-3.0/library/src/test/java/io/opentelemetry/instrumentation/failsafe/v3_0/FailsafeTelemetryTest.java

@@ -101,7 +101,7 @@ void captureRetryPolicyMetrics() {
         failsafeTelemetry.createRetryPolicy(userRetryPolicy, "testing");
 
     // when
-    for (int i = 0; i <= 3; i++) {
+    for (int i = 0; i <= 4; i++) {
       int temp = i;
       AtomicInteger retry = new AtomicInteger(0);
       Failsafe.with(instrumentedRetryPolicy)
@@ -131,7 +131,7 @@ void captureRetryPolicyMetrics() {
         .anyMatch(
             p ->
                 p.getAttributes().equals(buildExpectedRetryPolicyAttributes("failure"))
-                    && p.getValue() == 1);
+                    && p.getValue() == 2);
     assertThat(executionCountMetric.getPoints())
         .anyMatch(
             p ->
@@ -153,15 +153,15 @@ void captureRetryPolicyMetrics() {
                     && p.getMin() == 1
                     && p.getMax() == 3
                     && p.getAttributes().equals(buildExpectedRetryPolicyAttributes("success"))
-                    && Arrays.equals(p.getCounts().toArray(), new Long[] {1L, 1L, 1L, 0L}));
+                    && Arrays.equals(p.getCounts().toArray(), new Long[] {1L, 1L, 1L, 0L, 0L}));
     assertThat(pointData)
         .anyMatch(
             p ->
-                p.getCount() == 1
+                p.getCount() == 2
                     && p.getMin() == 3
                     && p.getMax() == 3
                     && p.getAttributes().equals(buildExpectedRetryPolicyAttributes("failure"))
-                    && Arrays.equals(p.getCounts().toArray(), new Long[] {0L, 0L, 1L, 0L}));
+                    && Arrays.equals(p.getCounts().toArray(), new Long[] {0L, 0L, 2L, 0L, 0L}));
   }
 
   private static Consumer<LongPointAssert> buildCircuitBreakerAssertion(