This is an automated email from the ASF dual-hosted git repository.

zhangliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new 4959d2317f7 Add javadoc on shardingsphere-firebird-protocol module if 
missing in class (#35995)
4959d2317f7 is described below

commit 4959d2317f7304311967b6f61834edd82350a5c1
Author: Liang Zhang <[email protected]>
AuthorDate: Mon Jul 14 23:45:07 2025 +0800

    Add javadoc on shardingsphere-firebird-protocol module if missing in class 
(#35995)
---
 .../constant/buffer/FirebirdParameterBuffer.java   |  3 ++
 .../buffer/FirebirdParameterBufferType.java        |  3 ++
 .../type/FirebirdDatabaseParameterBufferType.java  |  3 ++
 .../FirebirdTransactionParameterBufferType.java    |  3 ++
 .../constant/protocol/FirebirdProtocol.java        |  3 ++
 .../constant/protocol/FirebirdProtocolVersion.java |  3 ++
 .../firebird/err/FirebirdStatusVector.java         |  4 +-
 .../admin/FirebirdUnsupportedCommandPacket.java    |  2 +-
 .../command/query/info/FirebirdInfoPacketType.java |  3 ++
 .../database/FirebirdDatabaseInfoPacketType.java   |  3 ++
 .../database/FirebirdDatabaseInfoReturnPacket.java |  2 +-
 .../info/type/sql/FirebirdSQLInfoPacketType.java   |  3 ++
 .../info/type/sql/FirebirdSQLInfoReturnPacket.java |  2 +-
 .../statement/FirebirdAllocateStatementPacket.java |  9 ++--
 .../statement/FirebirdFetchStatementPacket.java    |  9 ++--
 .../statement/FirebirdFreeStatementPacket.java     |  9 ++--
 .../protocol/util/FirebirdDateTimeUtil.java        |  3 ++
 .../FirebirdPrepareStatementReturnPacket.java      |  8 ++--
 .../generic/FirebirdGenericResponsePacket.java     | 51 ++++++++++++----------
 .../packet/handshake/FirebirdAcceptDataPacket.java |  2 +-
 20 files changed, 78 insertions(+), 50 deletions(-)

diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/FirebirdParameterBuffer.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/FirebirdParameterBuffer.java
index 4e9bbb210eb..0ffd7350208 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/FirebirdParameterBuffer.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/FirebirdParameterBuffer.java
@@ -27,6 +27,9 @@ import java.util.HashMap;
 import java.util.Map;
 import java.util.function.Function;
 
+/**
+ * Firebird parameter buffer.
+ */
 @RequiredArgsConstructor
 public final class FirebirdParameterBuffer {
     
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/FirebirdParameterBufferType.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/FirebirdParameterBufferType.java
index 9f6f7a7abcf..ae0158c344b 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/FirebirdParameterBufferType.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/FirebirdParameterBufferType.java
@@ -19,6 +19,9 @@ package 
org.apache.shardingsphere.db.protocol.firebird.constant.buffer;
 
 import 
org.apache.shardingsphere.db.protocol.firebird.constant.FirebirdValueFormat;
 
+/**
+ * Firebird parameter buffer type.
+ */
 public interface FirebirdParameterBufferType {
     
     /**
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/type/FirebirdDatabaseParameterBufferType.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/type/FirebirdDatabaseParameterBufferType.java
index 24badeb20bc..fc1ef272097 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/type/FirebirdDatabaseParameterBufferType.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/type/FirebirdDatabaseParameterBufferType.java
@@ -27,6 +27,9 @@ import 
org.apache.shardingsphere.db.protocol.firebird.constant.buffer.FirebirdPa
 import java.util.HashMap;
 import java.util.Map;
 
+/**
+ * Firebird database parameter buffer type.
+ */
 @RequiredArgsConstructor
 @Getter
 public enum FirebirdDatabaseParameterBufferType implements 
FirebirdParameterBufferType {
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/type/FirebirdTransactionParameterBufferType.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/type/FirebirdTransactionParameterBufferType.java
index 565470d91db..71a39943a0b 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/type/FirebirdTransactionParameterBufferType.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/buffer/type/FirebirdTransactionParameterBufferType.java
@@ -27,6 +27,9 @@ import 
org.apache.shardingsphere.db.protocol.firebird.constant.buffer.FirebirdPa
 import java.util.HashMap;
 import java.util.Map;
 
+/**
+ * Firebird database parameter buffer type.
+ */
 @RequiredArgsConstructor
 @Getter
 public enum FirebirdTransactionParameterBufferType implements 
FirebirdParameterBufferType {
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/protocol/FirebirdProtocol.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/protocol/FirebirdProtocol.java
index a00463540b7..3ab8ea3cb5c 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/protocol/FirebirdProtocol.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/protocol/FirebirdProtocol.java
@@ -21,6 +21,9 @@ import io.netty.buffer.ByteBuf;
 import lombok.Getter;
 import 
org.apache.shardingsphere.db.protocol.firebird.constant.FirebirdArchType;
 
+/**
+ * Firebird protocol.
+ */
 @Getter
 public final class FirebirdProtocol {
     
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/protocol/FirebirdProtocolVersion.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/protocol/FirebirdProtocolVersion.java
index 28fa6420bdf..907dd4fbe48 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/protocol/FirebirdProtocolVersion.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/constant/protocol/FirebirdProtocolVersion.java
@@ -24,6 +24,9 @@ import lombok.RequiredArgsConstructor;
 import java.util.HashMap;
 import java.util.Map;
 
+/**
+ * Firebird protocol version.
+ */
 @RequiredArgsConstructor
 @Getter
 public enum FirebirdProtocolVersion {
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/err/FirebirdStatusVector.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/err/FirebirdStatusVector.java
index b0edeb236b0..d30ced4d950 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/err/FirebirdStatusVector.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/err/FirebirdStatusVector.java
@@ -37,9 +37,7 @@ public final class FirebirdStatusVector extends 
FirebirdPacket {
     private String errorMessage;
     
     public FirebirdStatusVector(final SQLException ex) {
-        gdsCode = ex.getErrorCode() >= ISCConstants.isc_arith_except
-                ? ex.getErrorCode()
-                : ISCConstants.isc_random;
+        gdsCode = ex.getErrorCode() >= ISCConstants.isc_arith_except ? 
ex.getErrorCode() : ISCConstants.isc_random;
         String rawMessage = ex.getMessage();
         int idx = rawMessage.indexOf(';');
         String message = idx >= 0 ? rawMessage.substring(idx + 1).trim() : 
rawMessage;
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/admin/FirebirdUnsupportedCommandPacket.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/admin/FirebirdUnsupportedCommandPacket.java
index f74459e21db..792e8bcce40 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/admin/FirebirdUnsupportedCommandPacket.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/admin/FirebirdUnsupportedCommandPacket.java
@@ -26,8 +26,8 @@ import 
org.apache.shardingsphere.db.protocol.firebird.payload.FirebirdPacketPayl
 /**
  * Unsupported command packet for Firebird.
  */
-@Getter
 @RequiredArgsConstructor
+@Getter
 public final class FirebirdUnsupportedCommandPacket extends 
FirebirdCommandPacket {
     
     private final FirebirdCommandPacketType code;
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/FirebirdInfoPacketType.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/FirebirdInfoPacketType.java
index 74582c40281..aa9c7f11c95 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/FirebirdInfoPacketType.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/FirebirdInfoPacketType.java
@@ -17,6 +17,9 @@
 
 package 
org.apache.shardingsphere.db.protocol.firebird.packet.command.query.info;
 
+/**
+ * Firebird info packet type.
+ */
 public interface FirebirdInfoPacketType {
     
     /**
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/database/FirebirdDatabaseInfoPacketType.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/database/FirebirdDatabaseInfoPacketType.java
index d7c55a56111..fde087065c8 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/database/FirebirdDatabaseInfoPacketType.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/database/FirebirdDatabaseInfoPacketType.java
@@ -26,6 +26,9 @@ import 
org.apache.shardingsphere.db.protocol.firebird.payload.FirebirdPacketPayl
 import java.util.HashMap;
 import java.util.Map;
 
+/**
+ * Firebird database info packet type.
+ */
 @RequiredArgsConstructor
 @Getter
 public enum FirebirdDatabaseInfoPacketType implements FirebirdInfoPacketType {
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/database/FirebirdDatabaseInfoReturnPacket.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/database/FirebirdDatabaseInfoReturnPacket.java
index c7d5310fbbe..694e343ea61 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/database/FirebirdDatabaseInfoReturnPacket.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/database/FirebirdDatabaseInfoReturnPacket.java
@@ -32,8 +32,8 @@ import java.util.List;
 /**
  * Database info return data packet for Firebird.
  */
-@Getter
 @RequiredArgsConstructor
+@Getter
 public final class FirebirdDatabaseInfoReturnPacket extends FirebirdPacket {
     
     private static final int SQL_DIALECT = 3;
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/sql/FirebirdSQLInfoPacketType.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/sql/FirebirdSQLInfoPacketType.java
index 5557c86d9e6..cb5c5accc6c 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/sql/FirebirdSQLInfoPacketType.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/sql/FirebirdSQLInfoPacketType.java
@@ -26,6 +26,9 @@ import 
org.apache.shardingsphere.db.protocol.firebird.payload.FirebirdPacketPayl
 import java.util.HashMap;
 import java.util.Map;
 
+/**
+ * Firebird SQL info packet type.
+ */
 @RequiredArgsConstructor
 @Getter
 public enum FirebirdSQLInfoPacketType implements FirebirdInfoPacketType {
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/sql/FirebirdSQLInfoReturnPacket.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/sql/FirebirdSQLInfoReturnPacket.java
index 6f738b473fe..1b571c4b5c5 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/sql/FirebirdSQLInfoReturnPacket.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/info/type/sql/FirebirdSQLInfoReturnPacket.java
@@ -30,8 +30,8 @@ import java.util.List;
 /**
  * SQL info return data packet for Firebird.
  */
-@Getter
 @RequiredArgsConstructor
+@Getter
 public final class FirebirdSQLInfoReturnPacket extends FirebirdPacket {
     
     private final List<FirebirdInfoPacketType> infoItems;
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdAllocateStatementPacket.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdAllocateStatementPacket.java
index ff6bee60468..56acf720ba0 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdAllocateStatementPacket.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdAllocateStatementPacket.java
@@ -34,11 +34,6 @@ public final class FirebirdAllocateStatementPacket extends 
FirebirdCommandPacket
         databaseId = payload.readInt4();
     }
     
-    @Override
-    protected void write(final FirebirdPacketPayload payload) {
-        
-    }
-    
     /**
      * Get length of packet.
      *
@@ -48,4 +43,8 @@ public final class FirebirdAllocateStatementPacket extends 
FirebirdCommandPacket
     public static int getLength(final FirebirdPacketPayload payload) {
         return 8;
     }
+    
+    @Override
+    protected void write(final FirebirdPacketPayload payload) {
+    }
 }
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdFetchStatementPacket.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdFetchStatementPacket.java
index 182c0326bb5..6c97ccfec51 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdFetchStatementPacket.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdFetchStatementPacket.java
@@ -89,11 +89,6 @@ public final class FirebirdFetchStatementPacket extends 
FirebirdCommandPacket {
         }
     }
     
-    @Override
-    protected void write(final FirebirdPacketPayload payload) {
-        
-    }
-    
     /**
      * Get length of packet.
      *
@@ -105,4 +100,8 @@ public final class FirebirdFetchStatementPacket extends 
FirebirdCommandPacket {
         length += payload.getBufferLength(length);
         return length + 8;
     }
+    
+    @Override
+    protected void write(final FirebirdPacketPayload payload) {
+    }
 }
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdFreeStatementPacket.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdFreeStatementPacket.java
index 1c07e09fe90..6f7f33aea34 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdFreeStatementPacket.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/FirebirdFreeStatementPacket.java
@@ -43,11 +43,6 @@ public final class FirebirdFreeStatementPacket extends 
FirebirdCommandPacket {
         option = payload.readInt4();
     }
     
-    @Override
-    protected void write(final FirebirdPacketPayload payload) {
-        
-    }
-    
     /**
      * Get length of packet.
      *
@@ -57,4 +52,8 @@ public final class FirebirdFreeStatementPacket extends 
FirebirdCommandPacket {
     public static int getLength(final FirebirdPacketPayload payload) {
         return 12;
     }
+    
+    @Override
+    protected void write(final FirebirdPacketPayload payload) {
+    }
 }
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/execute/protocol/util/FirebirdDateTimeUtil.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/execute/protocol/util/FirebirdDateTimeUtil.java
index 19c70442174..affc70b88d9 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/execute/protocol/util/FirebirdDateTimeUtil.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/execute/protocol/util/FirebirdDateTimeUtil.java
@@ -23,6 +23,9 @@ import lombok.Setter;
 import java.sql.Timestamp;
 import java.time.LocalDateTime;
 
+/**
+ * Firebird date and time utility class.
+ */
 @Getter
 @Setter
 public final class FirebirdDateTimeUtil {
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/prepare/FirebirdPrepareStatementReturnPacket.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/prepare/FirebirdPrepareStatementReturnPacket.java
index 749b7dcffbe..845dec37215 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/prepare/FirebirdPrepareStatementReturnPacket.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/command/query/statement/prepare/FirebirdPrepareStatementReturnPacket.java
@@ -33,16 +33,16 @@ import java.util.List;
 /**
  * Firebird prepare statement return data packet.
  */
-@Getter
 @RequiredArgsConstructor
+@Getter
+@Setter
 public final class FirebirdPrepareStatementReturnPacket extends FirebirdPacket 
{
     
-    @Setter
     private FirebirdSQLInfoReturnValue type;
     
-    private List<FirebirdReturnColumnPacket> describeSelect = new 
ArrayList<>();
+    private final List<FirebirdReturnColumnPacket> describeSelect = new 
ArrayList<>();
     
-    private List<FirebirdReturnColumnPacket> describeBind = new ArrayList<>();
+    private final List<FirebirdReturnColumnPacket> describeBind = new 
ArrayList<>();
     
     @Override
     protected void write(final FirebirdPacketPayload payload) {
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/generic/FirebirdGenericResponsePacket.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/generic/FirebirdGenericResponsePacket.java
index fcb3e50604d..7d0e0c053f8 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/generic/FirebirdGenericResponsePacket.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/generic/FirebirdGenericResponsePacket.java
@@ -18,7 +18,6 @@
 package org.apache.shardingsphere.db.protocol.firebird.packet.generic;
 
 import lombok.Getter;
-import lombok.NoArgsConstructor;
 import org.apache.shardingsphere.db.protocol.firebird.err.FirebirdStatusVector;
 import org.apache.shardingsphere.db.protocol.firebird.packet.FirebirdPacket;
 import 
org.apache.shardingsphere.db.protocol.firebird.packet.command.FirebirdCommandPacketType;
@@ -30,7 +29,6 @@ import java.sql.SQLException;
  * Generic response data packet for Firebird.
  */
 @Getter
-@NoArgsConstructor
 public final class FirebirdGenericResponsePacket extends FirebirdPacket {
     
     private int handle;
@@ -41,6 +39,15 @@ public final class FirebirdGenericResponsePacket extends 
FirebirdPacket {
     
     private FirebirdStatusVector statusVector;
     
+    /**
+     * Get generic response packet.
+     *
+     * @return generic response packet
+     */
+    public static FirebirdGenericResponsePacket getPacket() {
+        return new FirebirdGenericResponsePacket();
+    }
+    
     /**
      * Set handle value.
      *
@@ -85,6 +92,24 @@ public final class FirebirdGenericResponsePacket extends 
FirebirdPacket {
         return this;
     }
     
+    /**
+     * Extract error code from status vector.
+     *
+     * @return error code
+     */
+    public int getErrorCode() {
+        return statusVector == null ? -1 : statusVector.getGdsCode();
+    }
+    
+    /**
+     * Extract error message from status vector.
+     *
+     * @return error message
+     */
+    public String getErrorMessage() {
+        return statusVector == null ? "" : statusVector.getErrorMessage();
+    }
+    
     @Override
     protected void write(final FirebirdPacketPayload payload) {
         payload.writeInt4(FirebirdCommandPacketType.RESPONSE.getValue());
@@ -106,26 +131,4 @@ public final class FirebirdGenericResponsePacket extends 
FirebirdPacket {
             payload.getByteBuf().writeZero(4);
         }
     }
-    
-    public static FirebirdGenericResponsePacket getPacket() {
-        return new FirebirdGenericResponsePacket();
-    }
-    
-    /**
-     * Extract error code from status vector.
-     *
-     * @return error code
-     */
-    public int getErrorCode() {
-        return statusVector == null ? -1 : statusVector.getGdsCode();
-    }
-    
-    /**
-     * Extract error message from status vector.
-     *
-     * @return error message
-     */
-    public String getErrorMessage() {
-        return statusVector == null ? "" : statusVector.getErrorMessage();
-    }
 }
diff --git 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/handshake/FirebirdAcceptDataPacket.java
 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/handshake/FirebirdAcceptDataPacket.java
index 3989defcf64..d576db5f143 100644
--- 
a/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/handshake/FirebirdAcceptDataPacket.java
+++ 
b/db-protocol/firebird/src/main/java/org/apache/shardingsphere/db/protocol/firebird/packet/handshake/FirebirdAcceptDataPacket.java
@@ -28,8 +28,8 @@ import java.nio.charset.StandardCharsets;
 /**
  * Accept data packet for Firebird.
  */
-@Getter
 @RequiredArgsConstructor
+@Getter
 public final class FirebirdAcceptDataPacket extends FirebirdPacket {
     
     private final byte[] salt;

Reply via email to