Skip to content

Commit

Permalink
branch-3.0: [enhancement](log) logout tablename with temp partitions …
Browse files Browse the repository at this point in the history
…in backup (apache#42549)

Co-authored-by: Yongqiang YANG <[email protected]>
  • Loading branch information
github-actions[bot] and dataroaring authored Nov 7, 2024
1 parent c719357 commit 85f839f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -421,14 +421,14 @@ private void backup(Repository repository, Database db, BackupStmt stmt) throws
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()) {
Expand Down Expand Up @@ -669,7 +669,7 @@ public void checkAndFilterRestoreOlapTableExistInSnapshot(Map<String, BackupOlap
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()) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down

0 comments on commit 85f839f

Please sign in to comment.