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

Do we need the metric snapshot routes? #75

Open
diegoamc opened this issue Feb 17, 2016 · 2 comments
Open

Do we need the metric snapshot routes? #75

diegoamc opened this issue Feb 17, 2016 · 2 comments

Comments

@diegoamc
Copy link
Contributor

In what situations are we using the metric snapshot routes?

They need the metric snapshot id, but the user should not even know it.

If there are no use cases, maybe we should remove them.

@danielkza
Copy link
Contributor

In my opinion we do not need them, the existence of snapshots should be just an internal implementation detail, and their IDs won't even be made available anywhere.

@rafamanzo
Copy link
Member

Agreed on the removal.

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

No branches or pull requests

3 participants