This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-2.1 by this push:
new 97e9d4d9c0e [enhancement](log) logout tablename with temp partitions
in backup (#… (#43139)
97e9d4d9c0e is described below
commit 97e9d4d9c0e968e7312231fb85e678b1f8bb9971
Author: Yongqiang YANG <[email protected]>
AuthorDate: Mon Nov 11 23:14:53 2024 +0800
[enhancement](log) logout tablename with temp partitions in backup (#…
(#43139)
…42143)
---
fe/fe-core/src/main/java/org/apache/doris/backup/BackupHandler.java | 6 +++---
.../backup_restore/test_backup_restore_backup_temp_partition.groovy | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/backup/BackupHandler.java
b/fe/fe-core/src/main/java/org/apache/doris/backup/BackupHandler.java
index 49190acce1a..1318392d8aa 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/backup/BackupHandler.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/backup/BackupHandler.java
@@ -408,14 +408,14 @@ public class BackupHandler extends MasterDaemon
implements Writable {
try {
if (olapTbl.existTempPartitions()) {
ErrorReport.reportDdlException(ErrorCode.ERR_COMMON_ERROR,
- "Do not support backup table with temp
partitions");
+ "Do not support backup table " + olapTbl.getName()
+ " with temp partitions");
}
PartitionNames partitionNames = tblRef.getPartitionNames();
if (partitionNames != null) {
if (partitionNames.isTemp()) {
ErrorReport.reportDdlException(ErrorCode.ERR_COMMON_ERROR,
- "Do not support backup temp partitions");
+ "Do not support backup temp partitions in
table " + tblRef.getName());
}
for (String partName : partitionNames.getPartitionNames())
{
@@ -671,7 +671,7 @@ public class BackupHandler extends MasterDaemon implements
Writable {
if (partitionNames != null) {
if (partitionNames.isTemp()) {
ErrorReport.reportDdlException(ErrorCode.ERR_COMMON_ERROR,
- "Do not support restoring temporary partitions");
+ "Do not support restoring temporary partitions in
table " + tblName);
}
// check the selected partitions
for (String partName : partitionNames.getPartitionNames()) {
diff --git
a/regression-test/suites/backup_restore/test_backup_restore_backup_temp_partition.groovy
b/regression-test/suites/backup_restore/test_backup_restore_backup_temp_partition.groovy
index 7a99214ae46..61514bedd88 100644
---
a/regression-test/suites/backup_restore/test_backup_restore_backup_temp_partition.groovy
+++
b/regression-test/suites/backup_restore/test_backup_restore_backup_temp_partition.groovy
@@ -68,7 +68,7 @@ suite("test_backup_restore_backup_temp_partition",
"backup_restore") {
TO `${repoName}`
ON (${tableName})
"""
- exception "Do not support backup table with temp partitions"
+ exception "Do not support backup table ${tableName} with temp
partitions"
}
sql "DROP TABLE ${dbName}.${tableName} FORCE"
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
