-
Notifications
You must be signed in to change notification settings - Fork 413
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
Memory leak in expand_alternative_phrase_option #636
Comments
Ran some expander tests under valgrind, and am not seeing any memory leaks on the C side. The code complexity in expand_address is greater than I'd prefer, so depending on the input (language, script, transliteration) there are a number of different paths it can take. Not seeing anything immediately apparent in expand_alternative_phrase_option but would need a reproducible test case. It's best if you can reduce the problem to a single input. In order to conclude whether there's a leak in libpostal itself vs. somewhere else, it should be possible to run (without Java, just use the native cli program that builds as part of libpostal after running |
Hi we have a service that runs AddressExpander java binding. The pods get restarted due to OOM once in a while and we did some analysis on possible memory leak. It turns out that there might be memory leak in
![app-profiling 439](https://private-user-images.githubusercontent.com/22250698/258182923-fa64427f-355b-420b-a450-9d9f6299ed4d.gif?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTg5NTc5MTIsIm5iZiI6MTcxODk1NzYxMiwicGF0aCI6Ii8yMjI1MDY5OC8yNTgxODI5MjMtZmE2NDQyN2YtMzU1Yi00MjBiLWE0NTAtOWQ5ZjYyOTllZDRkLmdpZj9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA2MjElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNjIxVDA4MTMzMlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWY5OGUwNTUxYWZmZTVkYmNjNzk1MWJjODVhZTBlODIzNTFlMmM3ZDBiNmE5NzQzMzdkZWU4OWIzMzFiN2ZjMDkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.nk-Z0VEEosA1evnoWUYLvbhEidh_DTv0jFrbrB_j-Jw)
expand_alternative_phrase_option
function, as shown in the jemalloc analysis output hereAny idea how we can solve this issue?
The text was updated successfully, but these errors were encountered: