Backward compatibility, I think.

At least the "ms" one is fairly new, and I think we left the others to
avoid break configuration during upgrade.

On Fri, Jul 24, 2015 at 6:34 PM, Mayuresh Gharat
<gharatmayures...@gmail.com> wrote:
> I was thinking why we have 3 different configs for the same property (log
> retention)  :
>
> "log.retention.ms"
> "log.retention.minutes"
> "log.retention.hours"
>
> Why don't we only use the Milliseconds?
> There are other properties as well like log Jitter, LogRollTime which raise
> the same question in my mind.
>
> The reason I am asking this is that a user might misconfigure them like
> setting the log.retention.ms = 1 and log.retention.hours = 1.
>
> --
> -Regards,
> Mayuresh R. Gharat
> (862) 250-7125

Reply via email to