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

Use bson.json_util to flatten out events received when description in… #701

Closed
wants to merge 2 commits into from
Closed

Use bson.json_util to flatten out events received when description in… #701

wants to merge 2 commits into from

Conversation

whardier
Copy link

…formation is present (replica sets.. possibly sharding information). Ensure that command is stringy when attempting to concatenate strings. Suggest use fstrings if older pythons can be depreciated

…formation is present (replica sets.. possibly sharding information). Ensure that command is stringy when attempting to concatenate strings. Suggest use fstrings if older pythons can be depreciated
@whardier whardier requested a review from a team May 17, 2020 03:54
…formation is present (replica sets.. possibly sharding information). Ensure that command is stringy when attempting to concatenate strings. Suggest use fstrings if older pythons can be depreciated
@whardier whardier closed this May 18, 2020
srikanthccv pushed a commit to srikanthccv/opentelemetry-python that referenced this pull request Nov 1, 2020
…ry#701)

* feat: add jaeger http trace format (open-telemetry#696)

* feat: add jaeger http trace format (open-telemetry#696)

* feat: add jaeger http trace format (open-telemetry#696)

* feat: add jaeger http trace format (open-telemetry#696)

* feat: add jaeger http trace format (open-telemetry#696)

* feat: add jaeger http trace format (open-telemetry#696)

* fix: we should set sampled\unsampled via flag

* fix: we should set sampled\unsampled via flag

* fix: flags should be converted to hex, not decimal

* feat: create new package for propagation jaeger

* fix: remove unused dependencies, correct readme header, moved out jaeger from core index.ts

* fix: added jaeger keyword

* fix: remove comma

* docs: replace NodeTracer with NodeTracerRegistry

* fix: added missing jaeger keyword to exporter-jaeger

* fix: remove test for browser

* fix: remove yarn for browser

* fix: use same naming style as other packages

* feat: added index.ts and version.ts, revert test for browser

* fix: tests added index-webpack.ts

* test: add test with span generated by jaeger client

* fix: apply review changes

* fix: move out from sub dirs

* docs: use common language for docs

* fix: test script fix

Co-authored-by: Uladzislau Kiva <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant