jedcunningham commented on code in PR #29058:
URL: https://github.com/apache/airflow/pull/29058#discussion_r1092214629


##########
docs/apache-airflow/howto/usage-cli.rst:
##########
@@ -217,6 +217,17 @@ You can use the ``--dry-run`` option to print the row 
counts in the primary tabl
 
 By default, ``db clean`` will archive purged rows in tables of the form 
``_airflow_deleted__<table>__<timestamp>``.  If you don't want the data 
preserved in this way, you may supply argument ``--skip-archive``.
 
+Export the purged records from the archive tables
+-------------------------------------------------
+The ``db export-cleaned`` command exports the contents of the archived tables, 
created by the ``db clean`` command,
+to a specified format, by default to a CSV file. The exported file will 
contain the records that were purged from the
+primary tables during the ``db clean`` process.
+
+You can specify the export format using ``--export-format`` option. The 
default format is csv.

Review Comment:
   Let me rephrase, as I have the same question (I think).
   
   ``export-format`` isn't required, but also has no default. So you must pass 
`csv`, and it's also the only option.
   
   I think having `csv` be the actual default is okay, but we need to mark it 
as such (then we wouldn't need to mark it as required). We should also list out 
the valid options here, which is only csv today.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to