yujunjun/datahub
2
0
Fork 0
mirror of https://github.com/datahub-project/datahub.git synced 2025-08-01 22:05:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(cli/delete): include aspect name in dry-run message (#7664)

Browse Source
This commit is contained in:
Harshal Sheth 2023-03-23 10:55:33 +05:30 committed by GitHub
parent 3b519924e8
commit cf40aecb84
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
metadata-ingestion/src/datahub/cli/delete_cli.py
View File

@ -432,7 +432,12 @@ def _delete_one_urn(
deletion_result.num_records = rows_affected deletion_result.num_records = rows_affected
deletion_result.num_timeseries_records = ts_rows_affected deletion_result.num_timeseries_records = ts_rows_affected
else: else:
logger.info(f"[Dry-run] Would hard-delete {urn} {soft_delete_msg}") if aspect_name:
logger.info(
f"[Dry-run] Would hard-delete aspect {aspect_name} of {urn} {soft_delete_msg}"
)
else:
logger.info(f"[Dry-run] Would hard-delete {urn} {soft_delete_msg}")
deletion_result.num_records = ( deletion_result.num_records = (
UNKNOWN_NUM_RECORDS # since we don't know how many rows will be affected UNKNOWN_NUM_RECORDS # since we don't know how many rows will be affected
) )