luwei16 commented on code in PR #66191:
URL: https://github.com/apache/doris/pull/66191#discussion_r3712685782
##########
fe/fe-core/src/main/java/org/apache/doris/catalog/Tablet.java:
##########
@@ -104,6 +104,10 @@ public TabletHealth() {
@SerializedName(value = "id")
protected long id;
+ // Cross-link for a split base/row-binlog tablet pair. Legacy images do not
+ // contain this field and therefore keep the non-positive "not aligned"
value.
+ @SerializedName(value = "ati")
+ protected long alignedTabletId = -1L;
Review Comment:
This introduces a second, incompatible representation of the base/row-binlog
tablet relationship. PR #65810 already persists directional links:
`base.rowBinlogTabletId` and `companion.rowBinlogBaseTabletId`, and its
creation, sink, and report paths use those fields. This PR instead adds
`alignedTabletId` and makes the locality scheduler depend on it. When the two
changes are combined, tablets created by #65810 leave `alignedTabletId` unset,
so the scheduler cannot resolve the pair. Please reuse #65810’s directional
fields as the canonical model, or provide an explicit compatibility/migration
layer instead of introducing a parallel persisted field.
--
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]
