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

exometer_report:unsubscribe doesn't work #51

Open
surik opened this issue Dec 17, 2015 · 2 comments
Open

exometer_report:unsubscribe doesn't work #51

surik opened this issue Dec 17, 2015 · 2 comments
Labels

Comments

@surik
Copy link
Contributor

surik commented Dec 17, 2015

Because here key is used without retry_failed_metrics

@uwiger
Copy link
Member

uwiger commented Dec 17, 2015

Ack. The retry_failed_metrics value really shouldn't have been part of the key in the first place. Since it is, I guess the unsubscribe_() function will need to use ets:select() with wildcards rather than ets:lookup().

@tolbrino
Copy link
Contributor

Or the option could be moved from the key to the subscriber record, in which case the lookup would remain reasonable.

@tolbrino tolbrino added the bug label Dec 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants