-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
refactor multiple parsing of same log file #529
Comments
@sebres I wondered if 0.10 doesn't do smth like that already? ;-) |
Unfortunately not :( But it was planned, because as I made an observer module (ban-time-incr), which thread work totally event driven, I've realized also that we can and should separate the monitoring functionality to different thread pool, that will notify action/filter pool doing the processing actions only (banning, unbanning etc.) I don't think, that it is currently a great problem, as long as the monitored log-lines still not washed out from system cache, but I'll nevertheless appreciate if we have done such refactoring. Maybe together with #1379 (or after it will be ready)? |
Users are using the same log file like mail.log for multiple jails. It may make sense to move all the time parsing there perhaps FileContainer.readline (with some per jail position).
The text was updated successfully, but these errors were encountered: