xuxueli/xxl-job

XxlJobAdminConfig bug?

john1337 opened this issue · 2 comments

john1337 commented 
public int getLogretentiondays() {
    if (logretentiondays < 7) {
        return -1;  // Limit greater than or equal to 7, otherwise close
    }
    return logretentiondays;
}

该方法注释与实际意图明显不一致，代码显示的是logretentiondays 配置若小于7则不清理日志，这有问题吧，比如就是要保存日志短一点，比如保留3天，上面的代码将导致日志清理任务永远不执行
image

zhipingwu commented

注释需要改改，改为make sure greater than 7 days, otherwise keep all log

john1337 commented

注释需要改改，改为make sure greater than 7 days, otherwise keep all log

那可能应用场景不一样，日志表很大时启停任务接口耗时就会增加到1秒以上，所以我自己的应用场景是更短的清除周期