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

Package structure for statsd should follow current accessM paradigms #62

Open
blast-hardcheese opened this issue Nov 28, 2020 · 1 comment

Comments

@blast-hardcheese
Copy link

My expectation when looking at zio.metrics.statsd was to find only the accessM proxy methods to whatever underlying layer is eventually supplied, the Has[???] type, and the layer constructors (clients, objects, etc).

I expect that moving everything around to what I think would be common idioms, set out by things like zio-logging and zio's built-in layers, should be coordinated with a ABI-breaking release, but I'm raising this issue just to start the discussion.

@toxicafunk
Copy link
Member

Hi @blast-hardcheese,

Thanks for your input and sorry for the late reply. There is a lot of work related to statsd reporting on zio-zmx, in particular this PR: zio/zio-zmx#155

I suggest you can have a look and possibly add to the discussion there, then we can decide whether to bring the changes to zio-metrics or if zio-zmx cover your needs better.

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

2 participants