starocean999 commented on code in PR #65219:
URL: https://github.com/apache/doris/pull/65219#discussion_r3533244026
##########
fe/fe-core/src/main/java/org/apache/doris/clone/DynamicPartitionScheduler.java:
##########
@@ -296,23 +335,29 @@ private ArrayList<AddPartitionOp>
getAddPartitionOp(Database db, OlapTable olapT
int hotPartitionNum = dynamicPartitionProperty.getHotPartitionNum();
String storagePolicyName = dynamicPartitionProperty.getStoragePolicy();
+ // Partition naming uses the same `now` clock as border computation,
+ // so names and values are based on the same timezone (UTC for
TIMESTAMPTZ).
String nowPartitionPrevBorder =
DynamicPartitionUtil.getPartitionRangeString(
dynamicPartitionProperty, now, 0, partitionFormat);
String nowPartitionName = dynamicPartitionProperty.getPrefix()
- +
DynamicPartitionUtil.getFormattedPartitionName(dynamicPartitionProperty.getTimeZone(),
+ +
DynamicPartitionUtil.getFormattedPartitionName(borderTimeZone,
nowPartitionPrevBorder,
dynamicPartitionProperty.getTimeUnit());
+ // UTC-based lower bound of the current partition, used to identify the
+ // current partition by range in getHistoricalPartitions().
+ String currentUtcBorder = DynamicPartitionUtil.getPartitionRangeString(
+ dynamicPartitionProperty, now, 0, partitionFormat);
+ currentUtcBorder = convertToUtcTimestamp(partitionColumn,
currentUtcBorder, borderTimeZone);
for (; idx <= dynamicPartitionProperty.getEnd(); idx++) {
+ // Borders for both partition values and names: use now.
String prevBorder = DynamicPartitionUtil.getPartitionRangeString(
dynamicPartitionProperty, now, idx, partitionFormat);
String nextBorder = DynamicPartitionUtil.getPartitionRangeString(
dynamicPartitionProperty, now, idx + 1, partitionFormat);
- // Save original border for partition name (must not contain UTC
suffix)
+ // Save pre-conversion border for naming (UTC-based for
TIMESTAMPTZ).
String prevBorderForName = prevBorder;
- prevBorder = convertToUtcTimestamp(partitionColumn, prevBorder,
- dynamicPartitionProperty.getTimeZone());
- nextBorder = convertToUtcTimestamp(partitionColumn, nextBorder,
- dynamicPartitionProperty.getTimeZone());
+ prevBorder = convertToUtcTimestamp(partitionColumn, prevBorder,
borderTimeZone);
Review Comment:
partition by day is from 00:00:00 to 00:00:00 in old path
--
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]
