[KAFKA-4326] Refactor LogCleaner to remove duplicate log copying logic

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.10.2.0
• Assignee: Jason Gustafson
• Reporter: Jason Gustafson
• Created: 20/Oct/16 23:25
• Updated: 28/Oct/16 10:32
• Resolved: 28/Oct/16 10:32

Description

I think there's some code duplication in the log cleaner with respect to the copying of the log and re-compression after cleaning. We have similar logic already in ByteBufferMessageSet which we can potentially reuse. This improves encapsulation so that message format changes in the future become a little easier (since they touch fewer components).

Issue Links

• links to: GitHub Pull Request #2053