From cf40aecb84d13c06584b97ec6c4127ffc48ef28e Mon Sep 17 00:00:00 2001 From: Harshal Sheth Date: Thu, 23 Mar 2023 10:55:33 +0530 Subject: [PATCH] fix(cli/delete): include aspect name in dry-run message (#7664) --- metadata-ingestion/src/datahub/cli/delete_cli.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/metadata-ingestion/src/datahub/cli/delete_cli.py b/metadata-ingestion/src/datahub/cli/delete_cli.py index f74c5f4df9..2aadcfa8ae 100644 --- a/metadata-ingestion/src/datahub/cli/delete_cli.py +++ b/metadata-ingestion/src/datahub/cli/delete_cli.py @@ -432,7 +432,12 @@ def _delete_one_urn( deletion_result.num_records = rows_affected deletion_result.num_timeseries_records = ts_rows_affected 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 = ( UNKNOWN_NUM_RECORDS # since we don't know how many rows will be affected )