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

Is it possible to open new API to clear data? #48

Open
yuanchieh-cheng opened this issue Dec 16, 2020 · 3 comments
Open

Is it possible to open new API to clear data? #48

yuanchieh-cheng opened this issue Dec 16, 2020 · 3 comments

Comments

@yuanchieh-cheng
Copy link

After run prometheus client for a long time, the memory keep eating up.
It would be better if I could simply call a api to flush all the collected data.
Is it possible or any better solution to prevent out of memory trouble?

@miroswan
Copy link
Contributor

There is not. Based on the test coverage that exists currently, there are no memory leaks; we maintain a suite of tests that check for these in CI. However, that doesn't necessarily mean there is not a possible memory leak in places where the library is not tested. I believe the best path forward it to improve the test coverage such that we can prove with absolute certainty that no leaks exists. Until then, please be diligent about using tools like Valgrind to determine where your memory issues might exist.

@yuanchieh-cheng
Copy link
Author

yuanchieh-cheng commented Dec 18, 2020

Sorry for my bad description.
I think this it not memory leak issue. The memory keep eating up just because there is no way to clear the data and the memory would ultimately run out.
Or is there any memory control mechanism inside the library ?
What will happen if I keep add metrics data ?

When I read the prometheus documentation, I could set the expiration time or max storage size to prevent the memory or disk full issue.
That's why I ask for exposing a API to clear the data.

Actually I came from this issue coturn/coturn#666

@batusekerci
Copy link

Sorry for my bad description. I think this it not memory leak issue. The memory keep eating up just because there is no way to clear the data and the memory would ultimately run out. Or is there any memory control mechanism inside the library ? What will happen if I keep add metrics data ?

When I read the prometheus documentation, I could set the expiration time or max storage size to prevent the memory or disk full issue. That's why I ask for exposing a API to clear the data.

Actually I came from this issue coturn/coturn#666

Could you please explain, how did you solve this issue?

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