[KAFKA-3762] Log.loadSegments() should log the message in exception

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.10.0.0
• Fix Version/s: 0.10.1.0
• Component/s: core
• Labels: newbie
• Assignee: Ishita Mandhan
• Reporter: Jun Rao
• Created: 27/May/16 00:48
• Updated: 14/Jun/16 18:55
• Resolved: 14/Jun/16 18:54

Description

In Log.loadSegments(), we have the following code. It would be useful to log the error message in IllegalArgumentException.

if(indexFile.exists()) {
try

{ segment.index.sanityCheck() }

catch

{ case e: java.lang.IllegalArgumentException => warn("Found a corrupted index file, %s, deleting and rebuilding index...".format(indexFile.getAbsolutePath)) indexFile.delete() segment.recover(config.maxMessageSize) }

}

Issue Links

• links to: GitHub Pull Request #1480