[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #29058: Optionally export `airflow db clean` data to CSV files

2023-01-26 Thread via GitHub
ephraimbuddy commented on code in PR #29058: URL: https://github.com/apache/airflow/pull/29058#discussion_r1087901858 ## airflow/utils/db_cleanup.py: ## @@ -159,6 +170,14 @@ def _do_delete(*, query, orm_model, skip_archive, session): logger.debug("delete statement:\n%s",

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #29058: Optionally export `airflow db clean` data to CSV files

2023-01-25 Thread via GitHub
ephraimbuddy commented on code in PR #29058: URL: https://github.com/apache/airflow/pull/29058#discussion_r1086571130 ## airflow/cli/cli_parser.py: ## @@ -468,7 +468,18 @@ def string_lower_type(val): help="Don't preserve purged records in an archive table.",

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #29058: Optionally export `airflow db clean` data to CSV files

2023-01-24 Thread via GitHub
ephraimbuddy commented on code in PR #29058: URL: https://github.com/apache/airflow/pull/29058#discussion_r1085074195 ## airflow/cli/cli_parser.py: ## @@ -468,7 +468,18 @@ def string_lower_type(val): help="Don't preserve purged records in an archive table.",

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #29058: Optionally export `airflow db clean` data to CSV files

2023-01-24 Thread via GitHub
ephraimbuddy commented on code in PR #29058: URL: https://github.com/apache/airflow/pull/29058#discussion_r1085074195 ## airflow/cli/cli_parser.py: ## @@ -468,7 +468,18 @@ def string_lower_type(val): help="Don't preserve purged records in an archive table.",

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #29058: Optionally export `airflow db clean` data to CSV files

2023-01-24 Thread via GitHub
ephraimbuddy commented on code in PR #29058: URL: https://github.com/apache/airflow/pull/29058#discussion_r1085055480 ## airflow/utils/db_cleanup.py: ## @@ -159,6 +170,14 @@ def _do_delete(*, query, orm_model, skip_archive, session): logger.debug("delete statement:\n%s",

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #29058: Optionally export `airflow db clean` data to CSV files

2023-01-23 Thread via GitHub
ephraimbuddy commented on code in PR #29058: URL: https://github.com/apache/airflow/pull/29058#discussion_r1084591088 ## airflow/utils/db_cleanup.py: ## @@ -123,7 +126,15 @@ def _check_for_rows(*, query: Query, print_rows=False): return num_entities -def _do_delete(*,