Codecov Report

Patch has no changes to coverable lines.

📢 Thoughts on this report? Let us know!.

// From this:
meter.histogramBuilder("foo")
  .setAdvice(advice -> advice
    .setExplicitBucketBoundaries(...)
    .setAttributes(..));

// to something like this:
meter.histogramBuilder("foo")
  .setExplicitBucketBoundariesAdvice(...)
  .setAttributesAdvice(...);

the second one looks simpler, so 👍🤷‍♂️ (but no really strong feelings)

one thing that's nice about the single setAdvice() method is that it gives one place to explain the whole advice story, but since the advice story differs a bit per advice type, maybe that's not much of an advantage(?)

Yeah I'm kind of a partial to the API proposed here. Another reason to go with separate methods is that the pattern of passing in a consumer to configure parameters may not be intuitive for every user.

Another reason to go with separate methods is that the pattern of passing in a consumer to configure parameters may not be intuitive for every user.

ya, I've noticed confusion about the autoconfiguration builder which uses this pattern

@open-telemetry/java-approvers PTAL. Would like to get this in for the next release so we can stabilize the API in the following. Or, potentially stabilize in the next release with this updated API.