Altinity / clickhouse-backup / 21356674427
68%

DEFAULT BRANCH: master
Ran
Jobs 0
Files 0
Run time –
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
21356674427

push

github

Slach 
fix https://github.com/Altinity/clickhouse-backup/issues/1265

Смотрю на код - проблема в том, что `RestoreData` также вызывается напрямую из API в `pkg/server/server.go`, и там не передаётся `filteredTables`. Нужно исправить и этот вызов.

Также есть проблема: когда `filteredTables` равен `nil` (при вызове только `restore_data` без полного restore), нужно применить ту же фильтрацию.

pkg/server/server.go
```go
<<<<<<< SEARCH
	if err := b.RestoreData(ctx, backupName, backupMetadata, true, metadataPath, tablePattern, partitions, skipProjections, disks, version, replicatedCopyToDetached); err != nil {
=======
	if err := b.RestoreData(ctx, backupName, backupMetadata, true, metadataPath, tablePattern, partitions, skipProjections, disks, version, replicatedCopyToDetached, nil); err != nil {
>>>>>>> REPLACE
```

Но это не решает основную проблему. Когда `filteredTables` равен `nil` и используется `--skip-empty-tables`, фильтрация не применяется в `RestoreData`.

Лучше передать флаг `skipEmptyTables` вместо отфильтрованного списка:

pkg/backup/restore.go
```go
<<<<<<< SEARCH
	if dataOnly || (schemaOnly == dataOnly) {
		if err := b.RestoreData(ctx, backupName, backupMetadata, dataOnly, metadataPath, tablePattern, partitions, skipProjections, disks, version, replicatedCopyToDetached, tablesForRestore); err != nil {
			return err
		}
	}
=======
	if dataOnly || (schemaOnly == dataOnly) {
		if err := b.RestoreData(ctx, backupName, backupMetadata, dataOnly, metadataPath, tablePattern, partitions, skipProjections, disks, version, replicatedCopyToDetached, skipEmptyTables); err != nil {
			return err
		}
	}
>>>>>>> REPLACE
```

pkg/backup/restore.go
```go
<<<<<<< SEARCH
// RestoreData - restore data for tables matched by tablePattern from backupName
func (b *Backuper) RestoreData(ctx context.Context, backupName string, backupMetadata metadata.BackupMetadata, dataOnly bool, metadataPath, tablePattern string, partitions, skipProjections []string, disks []clickhouse.Di... (continued)
Source Files on build 21356674427
Detailed source file information is not available for this build.