morrySnow commented on code in PR #34910:
URL: https://github.com/apache/doris/pull/34910#discussion_r1604291224
##########
fe/fe-core/src/main/java/org/apache/doris/mtmv/MTMVMaxTimestampSnapshot.java:
##########
@@ -25,17 +25,17 @@
* so the update time is used instead of the version
*/
public class MTMVMaxTimestampSnapshot implements MTMVSnapshotIf {
- // partitionId corresponding to timestamp
- // The reason why both timestamp and partitionId are stored is to avoid
+ // partitionName corresponding to timestamp
+ // The reason why both timestamp and partitionName are stored is to avoid
// deleting the partition corresponding to timestamp
- @SerializedName("p")
- private long partitionId;
+ @SerializedName("pn")
+ private String partitionName;
Review Comment:
When we upgrade from earlier versions, this field will be null. will there
be any issues?
--
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
