[KAFKA-7781] Add validation check for Topic retention.ms property

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.2.0
• Assignee: Kamal Chandraprakash
• Reporter: Kamal Chandraprakash
• Created: 02/Jan/19 11:42
• Updated: 13/Jan/19 13:19
• Resolved: 13/Jan/19 13:19

Description

Using AdminClient#alterConfigs, topic retention.ms property can be assigned to a value less than -1. This leads to inconsistency while describing the topic configuration. We should not allow values less than -1. In server.properties, if log.retention.ms configured to a value less than zero, it's set as -1.

This doesn't create any issue in log segment deletion, as the condition for infinite log retention checks for value less than zero. To maintain consistency, we should add the validation check.

Issue Links

• links to: GitHub Pull Request #6082