-
Notifications
You must be signed in to change notification settings - Fork 81
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
[bug] replace mmdb database sometimes results in crash #171
Comments
I think I have to revise this one as I did a short test with two custom geoip databases that had a changing field content and during using copying the mmdb and a running logstash the values changed quite immediately, so logstash is getting aware of it. Sorry for the confusion. |
When I replace the mmdb under the hood of logstash it seems that it sometimes crashes logstash which should even not happen, so let me reopen this one again.
|
For all general issues, please provide the following details for fast resolution:
today there is a need of restarting logstash for reloading the Maxmind geoip databases
Referring to this comment:
#63 (comment)
I would like to raise this now as it would be good to have the possibility to reload the database without restarting logstash, and with that you can update the database and logstash would be quite immediately aware of it.
From implementation perspective there are existing mechanisms already that can be taken as a blueprint from the translate plugin:
https://github.com/logstash-plugins/logstash-filter-translate/blob/5cbb57c9a34865856de9ead577c19d42c4b75ade/lib/logstash/filters/dictionary/file.rb#L119
reload_dictionary
https://github.com/logstash-plugins/logstash-filter-translate/blob/5cbb57c9a34865856de9ead577c19d42c4b75ade/lib/logstash/filters/dictionary/file.rb#L128
needs_refresh
and then offer the 2 configurable settings
refresh_interval
and if needed
refresh_behaviour
www.elastic.co/guide/en/logstash/current/plugins-filters-translate.html
The text was updated successfully, but these errors were encountered: