morningman commented on code in PR #32980:
URL: https://github.com/apache/doris/pull/32980#discussion_r1580576119


##########
fe/fe-core/src/main/java/org/apache/doris/transaction/TransactionState.java:
##########
@@ -298,6 +299,16 @@ public SchemaInfo(OlapTable olapTable) {
     // table id -> schema info
     private Map<Long, SchemaInfo> txnSchemas = new HashMap<>();
 
+    @Getter
+    @Setter
+    private List<SubTransactionState> subTransactionStates;
+    @Getter
+    @SerializedName(value = "subTxnIdToTableCommitInfo")

Review Comment:
   Use short name, such as:
   ```suggestion
       @SerializedName(value = "sti")
   ```



##########
fe/fe-core/src/main/java/org/apache/doris/load/loadv2/LoadJobFinalOperation.java:
##########
@@ -38,13 +39,20 @@
  * It is used to edit the job final state.
  */
 public class LoadJobFinalOperation extends TxnCommitAttachment implements 
Writable {
+    @SerializedName(value = "id")
     private long id;
+    @SerializedName(value = "loadingStatus")
     private EtlStatus loadingStatus = new EtlStatus();
+    @SerializedName(value = "progress")
     private int progress;
+    @SerializedName(value = "loadStartTimestamp")

Review Comment:
   use short name. like `lst`.
   Same comment for all other fields



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to