Skip to content
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

allows continuation of observations #1076

Merged

Conversation

marcingrzejszczak
Copy link
Contributor

without this change we're not taking into account a parent observation into account. Nor do we set in the reactor context the current observation for the users to use

with this change whenever an observation is being created we put it into the reacto context under a well-known micrometer.observation key (for more information look at the ObservationThreadLocalAccessor class from micrometer-core)

without this change we're not taking into account a parent observation into account. Nor do we set in the reactor context the current observation for the users to use

with this change whenever an observation is being created we put it into the reacto context under a well-known micrometer.observation key (for more information look at the ObservationThreadLocalAccessor class from micrometer-core)

Signed-off-by: Marcin Grzejszczak <mgrzejszczak@vmware.com>
@OlegDokuka OlegDokuka changed the title Allows continuation of observations allows continuation of observations Oct 27, 2022
@OlegDokuka OlegDokuka merged commit 4e1d8b5 into rsocket:1.1.x Oct 27, 2022
@OlegDokuka
Copy link
Member

as discussed with the team we will roll it into 1.2.0 instead. We will revert this commit for 1.1.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants