New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sets up MicrometerCapabilities #754
Sets up MicrometerCapabilities #754
Conversation
@@ -45,7 +45,7 @@ public boolean processArgument(AnnotatedParameterContext context, Annotation ann | |||
MethodMetadata metadata = context.getMethodMetadata(); | |||
if (metadata.queryMapIndex() == null) { | |||
metadata.queryMapIndex(paramIndex); | |||
metadata.queryMapEncoded(SpringQueryMap.class.cast(annotation).encoded()); | |||
// metadata.queryMapEncoded(SpringQueryMap.class.cast(annotation).encoded()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is no longer there in 11.10-SNAPSHOT
@@ -243,8 +244,10 @@ protected static class MetricsConfiguration { | |||
|
|||
@Bean | |||
@ConditionalOnMissingBean | |||
public MicrometerCapability micrometerCapability(MeterRegistry meterRegistry) { | |||
return new MicrometerCapability(meterRegistry); | |||
public MicrometerCapability micrometerCapability(MeterRegistry meterRegistry, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm thinking if we should add flags so that users can switch back to the MicrometerCapability
if they want the old behavior.
d280d23
to
d8c9f2b
Compare
Closing as the corresponding feature has already been added. |
prerequisite - OpenFeign/feign#1760
sample - micrometer-metrics/micrometer-samples@7485ae5