Closed maciej-gromul closed 2 weeks ago
From the javadoc of Micrometer.observation()
:
The Observation covers the entire length of the sequence, from subscription to termination
In the reference documentation we also have the list of metrics created by Micrometer.metrics()
and by Micrometer.observation()
. The one for Observation
doesn't list the onNextDelay
notion. It might be worth documenting the difference in one form or another, so let's keep this one open despite not gaining much attention for over a year. Perhaps someone is eager to contribute. @maciej-gromul seems you already had some ideas, are you willing to enhance the reference docs with regards to what you suggest?
I think it's resolved with #3862
Documentation Issue
I believe it would be beneficial to write down the difference in usage tap operator with Micrometer.observation() in different stages of defined pipeline. For example below we get 1 sec difference just with simple flatmap. If we offload our tasks after first tap to another scheduler and concatmap instead of flatmap, we would get 10sec difference. It would be worth writing down that it actually awaits the completion of whole publisher defined just before that tap and it actually cares only about context of a flux publisher as whole in comparison to previous
Flux.just(1,2,3).metrics()
that would observe additionally time between onNext calls and define a metric for those. Now to achieve that we need to write manual observations for that.Improvement Suggestion
Define simple examples showing the importance of placement of the tap subscriptions. It would be nice as well to document what is the output schema of the metrics that will be generated like it had been previously pointing to micrometer page for
.metrics()
operator.