MINOR: Make data in FetchSnapshotRequest and FetchSnapshotRespponse private (apache#9820)

Reviewers: José Armando García Sancio <[email protected]>, David Jacot <[email protected]>

Author: dengziming

clients/src/main/java/org/apache/kafka/common/requests/FetchSnapshotRequest.java

@@ -16,20 +16,20 @@
  */
 package org.apache.kafka.common.requests;
 
-import java.nio.ByteBuffer;
-import java.util.Collections;
-import java.util.Optional;
-import java.util.function.UnaryOperator;
 import org.apache.kafka.common.TopicPartition;
 import org.apache.kafka.common.message.FetchSnapshotRequestData;
 import org.apache.kafka.common.message.FetchSnapshotResponseData;
 import org.apache.kafka.common.protocol.ApiKeys;
-import org.apache.kafka.common.protocol.ApiMessage;
 import org.apache.kafka.common.protocol.ByteBufferAccessor;
 import org.apache.kafka.common.protocol.Errors;
 
+import java.nio.ByteBuffer;
+import java.util.Collections;
+import java.util.Optional;
+import java.util.function.UnaryOperator;
+
 final public class FetchSnapshotRequest extends AbstractRequest {
-    public final FetchSnapshotRequestData data;
+    private final FetchSnapshotRequestData data;
 
     public FetchSnapshotRequest(FetchSnapshotRequestData data, short version) {
         super(ApiKeys.FETCH_SNAPSHOT, version);
@@ -46,7 +46,7 @@ public FetchSnapshotResponse getErrorResponse(int throttleTimeMs, Throwable e) {
     }
 
     @Override
-    public ApiMessage data() {
+    public FetchSnapshotRequestData data() {
         return data;
     }
 

clients/src/main/java/org/apache/kafka/common/requests/FetchSnapshotResponse.java

@@ -16,25 +16,24 @@
  */
 package org.apache.kafka.common.requests;
 
+import org.apache.kafka.common.TopicPartition;
+import org.apache.kafka.common.message.FetchSnapshotResponseData;
+import org.apache.kafka.common.protocol.ApiKeys;
+import org.apache.kafka.common.protocol.ByteBufferAccessor;
+import org.apache.kafka.common.protocol.Errors;
+
 import java.nio.ByteBuffer;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Optional;
 import java.util.function.UnaryOperator;
-import org.apache.kafka.common.TopicPartition;
-import org.apache.kafka.common.message.FetchSnapshotResponseData;
-import org.apache.kafka.common.protocol.ApiKeys;
-import org.apache.kafka.common.protocol.ApiMessage;
-import org.apache.kafka.common.protocol.ByteBufferAccessor;
-import org.apache.kafka.common.protocol.Errors;
 
 final public class FetchSnapshotResponse extends AbstractResponse {
-    public final FetchSnapshotResponseData data;
+    private final FetchSnapshotResponseData data;
 
     public FetchSnapshotResponse(FetchSnapshotResponseData data) {
         super(ApiKeys.FETCH_SNAPSHOT);
-
         this.data = data;
     }
 
@@ -63,7 +62,7 @@ public int throttleTimeMs() {
     }
 
     @Override
-    public ApiMessage data() {
+    public FetchSnapshotResponseData data() {
         return data;
     }