cadence-java-client icon indicating copy to clipboard operation
cadence-java-client copied to clipboard

Published 20 hours ago • verified publisher icon uber

Refactor context prop support to allow things like opentelemtry to work properly with the cadence sdk

Open AngerM-DD opened this issue 3 years ago • 4 comments

Refactor the context propagators to support OpenTelemetry and add an example OTEL propagator.

AngerM-DD avatar Jun 15 '21 16:06 AngerM-DD

I triggered a build for this PR but as with my other PR (https://github.com/uber/cadence-java-client/pull/619), this one is failing build too. Something is broken with Buildkite and we are trying to understand what the issue is but compileThrift step is failing for some reason. We don't know the exact reason yet but hoping to resolve it soon. Let us know if you can find out something to fix the issue.

demirkayaender avatar Jul 13 '21 05:07 demirkayaender

Some update here: The issue is related to Java11 and Alpine. We are looking into how we could fix it. All the PRs will be potentially blocked until the fix unfortunately.

demirkayaender avatar Jul 13 '21 23:07 demirkayaender

Fix is here: https://github.com/uber/cadence-java-client/pull/625

demirkayaender avatar Jul 14 '21 04:07 demirkayaender

rebased this PR on top of 3.5.0. Have to go through all the comments still

AngerM-DD avatar Oct 26 '21 22:10 AngerM-DD