Is there anyway to support multiple reporters?

[tony612]

If my understanding is right, only one reporter is supported in Elixometer at the moment, right? If so, is there anyway to support multiple ones?

[scohen]

Right now, no, elixometer only allows you to subscribe to the reporter defined in Application.get_env(:elixometer, :reporter).

If you want to submit a PR for adding this, it should be rather easy.

[tony612]

@scohen Got it, I'll try to solve this. I think this should be a useful feature 😁

[scohen]

@tony612 Yes, I agree.

I think it's as simple as allowing a list to be passed to :reporter and checking for that an iterating over it in https://github.com/pinterest/elixometer/blob/master/lib/elixometer.ex#L349

[tony612]

@scohen There's one problem: How about deprecating :reporter to support :reporters?

[scohen]

I don't know how I feel about that; soft deprecation is OK, but should emit a warning. Hard deprecation will break things for people (including us).

What are your thoughts? :reporters can take a single names or a list of names?

[tony612]

@scohen Yeah, I'm OK with soft deprecation and we can remove :reporter in a future version. I refer :reporters only accept a list of names, which is more clear.