-
Notifications
You must be signed in to change notification settings - Fork 73
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
Ref #2973 : Automatic reports cleaning #5
Ref #2973 : Automatic reports cleaning #5
Conversation
@Value("${rudder.batch.reportsCleaner.delete.TTL}") | ||
var reportCleanerDeleteTTL = 90 | ||
|
||
@Value("${rudder.batch.reportsCleaner.frequency}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be better to use values defined in object FrequencyBuilder for that, but I'm not sure it is possible: I believe annotation only takes static modifier, so perhaps a "final" will be needed for vals.
Just testing - @Normation/documentors should also be concerned by this change. |
@Normation/administrators : sorry for the spam! |
|
||
override val logger = ReportLogger | ||
private var currentState: CleanerState = IdleCleaner | ||
private var lastRun: DateTime = DateTime.now() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
By habits, use the visibility private[this]
…n I/O with reports database
…ogger, and better indentation
…minutes in reports cleaning
…c reports cleaning section
…utomaticReportsCleaning helper object instead
…tsClean*, add comments and correct comments
I close this pull request has it had been retargeted to master, see #9 instead |
…Save_popup Ust 4180/dev 4182 manage directive on save popup
Following ticket #2972, this update goal is to provide a way to automate reports cleaning.
You have to define in your property files the parameters of your reports cleaning process
this is linked to http://www.rudder-project.org/redmine/issues/2973
You can ignore the commit related to ticket #2972 #2974 #2976, as they are treated in #4 and they will disapear when the first pull request will be accepted