[KAFKA-1927] Replace requests in kafka.api with requests in org.apache.kafka.common.requests

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Assignee: Gwen Shapira
• Reporter: Jay Kreps
• Created: 08/Feb/15 00:53
• Updated: 03/Jan/19 01:36
• Resolved: 03/Jan/19 01:36

Description

The common package introduced a better way of defining requests using a new protocol definition DSL and also includes wrapper objects for these.

We should switch KafkaApis over to use these request definitions and consider the scala classes deprecated (we probably need to retain some of them for a while for the scala clients).

This will be a big improvement because
1. We will have each request now defined in only one place (Protocol.java)
2. We will have built-in support for multi-version requests
3. We will have much better error messages (no more cryptic underflow errors)

Issue Links

• blocks: KAFKA-1694 KIP-4: Command line and centralized operations
• is blocked by: KAFKA-1634 Improve semantics of timestamp in OffsetCommitRequests and update documentation
• is depended upon by: KAFKA-1931 Refactor server to use code in org.apache.kafka.common
• is duplicated by: KAFKA-2006 switch the broker server over to the new java protocol definitions
• is duplicated by: KAFKA-1486 Move all request/responses to use schema-utils
• is related to: KAFKA-2929 Migrate server side error mapping functionality

Sub-Tasks

• Refactor KafkaApis to be optimal for o.a.k.c requests: Open