Update export-backups-to-own-cloud-account.md

[linhgiang24]

The command to restore from backup is incorrect. The correct command should be:

RESTORE DATABASE test_backups_restored 
AS test_backups 
FROM S3('https://testchbackups.s3.amazonaws.com/backups/<uuid>', '<key id>', '<key secret>')

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
clickhouse-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 16, 2025 7:12pm

3 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
clickhouse-docs-jp	⬜️ Ignored (Inspect)			Apr 16, 2025 7:12pm
clickhouse-docs-ru	⬜️ Ignored (Inspect)			Apr 16, 2025 7:12pm
clickhouse-docs-zh	⬜️ Ignored (Inspect)			Apr 16, 2025 7:12pm

[linhgiang24]

Actually, as I was reviewing the document again, all the restore from backup commands are incorrect - not just the one that I fixed.

From public documentation:

BACKUP|RESTORE
  TABLE [db.]table_name [AS [db.]table_name_in_backup]
    [PARTITION[S] partition_expr [,...]] |
  DICTIONARY [db.]dictionary_name [AS [db.]name_in_backup] |
  DATABASE database_name [AS database_name_in_backup]
    [EXCEPT TABLES ...] |
  TEMPORARY TABLE table_name [AS table_name_in_backup] |
  VIEW view_name [AS view_name_in_backup]
  ALL TEMPORARY TABLES [EXCEPT ...] |
  ALL [EXCEPT ...] } [,...]
  [ON CLUSTER 'cluster_name']
  TO|FROM File('<path>/<filename>') | Disk('<disk_name>', '<path>/') | S3('<S3 endpoint>/<path>', '<Access key ID>', '<Secret access key>')
  [SETTINGS base_backup = File('<path>/<filename>') | Disk(...) | S3('<S3 endpoint>/<path>', '<Access key ID>', '<Secret access key>')]

the important part is

  DATABASE database_name [AS database_name_in_backup]

so for restore, you need to do the database_name in the service where the restore is performed AS the name of the database in the backup

[gingerwizard]

this is restoring test_backups (backed up above) AS a new database, this doesnt seem incorrect to me

[gingerwizard]

This seems counterintuitive, we need to test as the reference spec actually looks wrong to me.

[gingerwizard]

so the syntax isn't quite documented right, should be:

BACKUP|RESTORE
  TABLE [db.]table_name_in_backup [AS [db.]table_name_to_restore_to]

CREATE DATABASE test
USE test


CREATE TABLE test
(
    `x` String
)
ENGINE = MergeTree
ORDER BY tuple()


INSERT INTO test FORMAT Values


BACKUP DATABASE test TO S3('https://storage.googleapis.com/sql.clickhouse.com/backup/testy', 'GOOGN77RURDAYFDRHV5QPZ7B', '<HIDDEN>')

Query id: 93cd3d19-1df2-4af3-b8f2-afc5112fd897

   ┌─id───────────────────────────────────┬─status─────────┐
1. │ 57bc0809-bf12-4d47-806d-032bdef51d0d │ BACKUP_CREATED │
   └──────────────────────────────────────┴────────────────┘

1 row in set. Elapsed: 5.448 sec.




DROP DATABSE test;



RESTORE DATABASE test AS test_v2 FROM S3('https://storage.googleapis.com/sql.clickhouse.com/backup/testy', 'GOOGN77RURDAYFDRHV5QPZ7B', '<HIDDEN>')

Query id: d18fda77-9e93-4656-8e05-03970a95de47

   ┌─id───────────────────────────────────┬─status───┐
1. │ 739bde16-05bf-42b7-a6d5-84c8a14c3d70 │ RESTORED │
   └──────────────────────────────────────┴──────────┘

1 row in set. Elapsed: 7.773 sec.

play-eu-clickhouse-client.localdomain :) show databases

SHOW DATABASES

Query id: aa165b44-bab9-4a37-a448-0d60beb41a6c

   ┌─name───────────────┐
1. │ INFORMATION_SCHEMA │
2. │ default            │
3. │ information_schema │
4. │ system             │
5. │ test_v2            │
   └────────────────────┘

5 rows in set. Elapsed: 0.001 sec.

[gingerwizard]

The issue is the syntax is correct for BACKUP but not for RESTORE where its inverted.