[jira] [Created] (KAFKA-5030) Suggested clean-ups from message format V2 review

["Ismael Juma (JIRA)" <ji...@apache.org>]

Ismael Juma created KAFKA-5030:
----------------------------------

             Summary: Suggested clean-ups from message format V2 review
                 Key: KAFKA-5030
                 URL: https://issues.apache.org/jira/browse/KAFKA-5030
             Project: Kafka
          Issue Type: Sub-task
            Reporter: Ismael Juma


A few random clean-ups raised during the review of the message format V2 PR:

1. We should use magic or message format version consistently. It seems like we are moving towards the former.
2. We should consider moving Log.analyzeAndValidateRecords to LogValidator so that the validation logic is all in the same class.
3. MemoryRecords.toString should include batch information, it currently only includes records information.
4. Record.hasMagic should be hasCompatibleMagic to make the behaviour clearer. For version 2, it returns true if the passed magic is greater than or equal to 2.
5. Once we move to Java 8, move AbstractRecordBatch's methods to RecordBatch (i.e. default methods) and remove AbstractRecordBatch.
6. Consider moving some of the down-conversion logic from KafkaApis to FetchRequest so that it can be more easily unit testable.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)