(doris) branch branch-2.1 updated: [enhancement](log) logout tablename with temp partitions in backup (#… (#43139)

[dataroaring]

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 <98214048+dataroar...@users.noreply.github.com>
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: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org