(doris) branch master updated: [enhancement](log) logout tablename with temp partitions in backup (#42143)

[dataroaring]

This is an automated email from the ASF dual-hosted git repository.

dataroaring pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 34a3a076649 [enhancement](log) logout tablename with temp partitions 
in backup (#42143)
34a3a076649 is described below

commit 34a3a07664903ab9ea4e48e63ed81041d09fe1a9
Author: Yongqiang YANG <98214048+dataroar...@users.noreply.github.com>
AuthorDate: Mon Oct 28 11:53:24 2024 +0800

    [enhancement](log) logout tablename with temp partitions in backup (#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 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: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org