This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-3.0 by this push:
new 85f839f9852 branch-3.0: [enhancement](log) logout tablename with temp
partitions in backup (#42549)
85f839f9852 is described below
commit 85f839f985265f426c98aaa575a2d081a4d9cc14
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Thu Nov 7 14:35:16 2024 +0800
branch-3.0: [enhancement](log) logout tablename with temp partitions in
backup (#42549)
Co-authored-by: Yongqiang YANG
<[email protected]>
---
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 e6687a95ebe..282b549c5df 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
@@ -421,14 +421,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())
{
@@ -669,7 +669,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]
