Hi Denis,
from point of content of the name matching to the meaning I agree - but
I have doubts because the column name conflicts with the SQL type that
has the same same. As we use ORM this is mostly fine but other logic
running SQL on the DB might get into conflict if column name "time" is
not quoted. I see a set of pitfalls with the specific name therefore.
See: https://www.postgresql.org/docs/current/datatype-datetime.html
In this light I'd propose to keep it like the existing name (as
timestamp has the same problem which just came into my mind as alternative)
Jens
On 26.05.25 18:09, Ferruzzi, Dennis wrote:
rawwar is working on some front-end stuff for the Deadlines feature and has a
suggestion to rename the database field which holds the calculated timestamp
from deadline.deadline to deadline.time. This seems reasonable to me and
slightly less confusing, but I wanted to throw it out there for lazy consensus,
or you can approve https://github.com/apache/airflow/pull/51026 for less-lazy
agreement.
Currently the table `deadline` has the following columns:
dag_id
dagrun_id
deadline: calculated timestamp
callback
callback_kwargs
- ferruzzi
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@airflow.apache.org
For additional commands, e-mail: dev-h...@airflow.apache.org