jtmiclat commented on PR #35250:
URL: https://github.com/apache/airflow/pull/35250#issuecomment-1784432516
@hussein-awala Thank you for your comments!
I think both are good suggestions and pushed some changes to reflect them.
1. I changed the parameter to just columns with default values
2. I raised a system exit if there are invalid columns provided
I just want to note the default behavior of `airflow dag list` in my
implementation has changed a bit. Specifically the column names are now using
the fields of the the dag schema rather than previous column names. Do you
think we should alias these column names for backward compatibility?
Before
```
root@2cce825aa9b1:/opt/airflow# airflow dags list
dag_id | filepath | owner | paused
============+==========+=========+=======
my_dag_name | test.py | airflow | True
```
After
```
root@2cce825aa9b1:/opt/airflow# airflow dags list
dag_id | fileloc | owners | is_paused
============+=====================+=========+==========
my_dag_name | /files/dags/test.py | airflow | True
```
--
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
