[KAFKA-3087] Fix documentation for retention.ms property and update documentation for LogConfig.scala class

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Component/s: log
• Labels: documentation
• Assignee: Jay Kreps
• Reporter: Raju Bairishetti
• Created: 11/Jan/16 09:32
• Updated: 25/Feb/18 21:37
• Resolved: 16/Aug/17 12:29

Description

Log retention settings can be set it in broker and some properties can be overriden at topic level.

Property	Default	Server Default property	Description
retention.ms	7 days	log.retention.minutes	This configuration controls the maximum time we will retain a log before we will discard old log segments to free up space if we are using the "delete" retention policy. This represents an SLA on how soon consumers must read their data.

But retention.ms is in milli seconds not in minutes. So corresponding Server Default property should be log.retention.ms instead of log.retention.minutes.

It would be better if we mention the if the time age is in millis/minutes/hours in the documentation page and documenting in code as well (Right now, it is saying age in the code. We should specify the *age in time granularity).

Issue Links

• links to: GitHub Pull Request #772