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

Multi reporter does not work with Reporter and CachedReporter #97

Open
project0 opened this issue Apr 19, 2019 · 3 comments
Open

Multi reporter does not work with Reporter and CachedReporter #97

project0 opened this issue Apr 19, 2019 · 3 comments

Comments

@project0
Copy link

Its currently not possible to send metrics to different reporters if those are not all Reporters or either CachedReports.

This does not work, as it will report values only to the reporter, but not to the cached reporter:

	scope, closer := tally.NewRootScope(tally.ScopeOptions{
		Prefix: "my_service",
		Tags:   map[string]string{},
		Reporter:       r, 
		CachedReporter: cr, //conflicts with above line
		Separator:      promreporter.DefaultSeparator,
	}, 1*time.Second)

One Solution would be probably to emulate cachedReporter for none-cached reporter in the multi package.

@lukashes
Copy link

Hi. Any updates?

@project0
Copy link
Author

Not sure, feels like the project is getting abandoned. I have also simple open PR since a year which is not getting merged...

@yoonsio
Copy link

yoonsio commented Apr 6, 2022

Are multiple reporters supported without implementing a custom reporter?

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

No branches or pull requests

3 participants