GitHub user rajubairishetti opened a pull request:

    https://github.com/apache/kafka/pull/772

    KAFKA-3087: Fix retention.ms property documentation in config docs

    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*.
    


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/rajubairishetti/kafka KAFKA-3087

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/kafka/pull/772.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #772
    
----
commit 7ea672013f5d028986f869fc6bf6a5c409655ef7
Author: raju <r...@apache.org>
Date:   2016-01-14T08:53:04Z

    KAFKA-3087: Fix retention.ms property documentation in config docs

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to