Skip to content
This repository has been archived by the owner on Aug 23, 2023. It is now read-only.

implement jaeger sampling #1060

Closed
Dieterbe opened this issue Sep 21, 2018 · 1 comment · Fixed by #1341
Closed

implement jaeger sampling #1060

Dieterbe opened this issue Sep 21, 2018 · 1 comment · Fixed by #1341

Comments

@Dieterbe
Copy link
Contributor

@gouthamve correct me if i'm wrong but for our internal deployment we want RemotelyControlledSampler right?

@Dieterbe
Copy link
Contributor Author

Dieterbe commented Sep 25, 2018

i thought i posted this already here but i guess not.

I'ld like to explore this "the MT native way" (e.g. use MT's configs mechanism, and configure the jaeger client accordingly) and ideally do it that way.
rather then #1055 which bypasses MT's config system.

unless doing it the MT way is very clunky or timeconsuming, then we can do it goutham's way.

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

Successfully merging a pull request may close this issue.

1 participant