gavinchou commented on code in PR #65810:
URL: https://github.com/apache/doris/pull/65810#discussion_r3630724669
##########
fe/fe-core/src/main/java/org/apache/doris/catalog/Tablet.java:
##########
@@ -104,6 +103,10 @@ public TabletHealth() {
@SerializedName(value = "id")
protected long id;
+ // The aligned tablet for the base <-> row binlog tablet relationship:
+ // on a base tablet it holds the binlog tablet id, on a binlog tablet it
holds the base tablet id.
+ @SerializedName(value = "ati")
+ protected long alignedTabletId = -1;
Review Comment:
better using 2 separate fields to make it simple and clear, there is no
side-effect using fields like
`rowBinlogBaseTabletId` and `rowBinlogTabletId` it wont occupy extra space
when serializing, because only one can be set.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
