Based on the comment here(https://github.com/apache/airflow/pull/50957#discussion_r2103894496) , we do delete these records after expiration. So, we can call it expiration
> if user doesn't see it, then it is much less important (and it can be > private). What is the interface for the user? I don't think this is finalised yet. On 2025/05/27 20:38:27 Daniel Standish wrote: > Oh boy, well, as they say, naming and cache invalidation. But expiration, > that does not sound good. That sorta suggests "this is the time when this > deadline record expires" as in, we delete it after this date -- or > something. Deadline time seems better to me. > > Anyway. > > Does the user see any of this? We're talking about database columns -- but > if user doesn't see it, then it is much less important (and it can be > private). What is the interface for the user? > > > > On Tue, May 27, 2025 at 11:32 AM Aritra Basu <aritrabasu1...@gmail.com> > wrote: > > > I like expiration, works with and without the table name. +1 from me in > > that one. > > -- > > Regards, > > Aritra Basu > > > > On Tue, 27 May 2025, 11:40 pm Ferruzzi, Dennis, > > <ferru...@amazon.com.invalid> > > wrote: > > > > > Ive been tossing around a few ideas and I think the best alternative I > > > came up with was `deadline.expiration`. It's clear what it means and > > > doesn't feel too awkward in use. I'm also good with deadline_time I > > guess, > > > but `deadline.deadline_time` "feels" awkward IMHO > > > > > > > > > - ferruzzi > > > > > > > > > ________________________________ > > > From: Daniel Standish <daniel.stand...@astronomer.io.INVALID> > > > Sent: Tuesday, May 27, 2025 9:42 AM > > > To: dev@airflow.apache.org > > > Subject: RE: [EXT] [Lazy Concensus] Naming tweak in Deadlines table > > > > > > CAUTION: This email originated from outside of the organization. Do not > > > click links or open attachments unless you can confirm the sender and > > know > > > the content is safe. > > > > > > > > > > > > AVERTISSEMENT: Ce courrier électronique provient d’un expéditeur externe. > > > Ne cliquez sur aucun lien et n’ouvrez aucune pièce jointe si vous ne > > pouvez > > > pas confirmer l’identité de l’expéditeur et si vous n’êtes pas certain > > que > > > le contenu ne présente aucun risque. > > > > > > > > > > > > Generally speaking, it's not a huge problem when you use a reserved word > > in > > > a column name. You add backticks or whatever. And we don't write the > > sql > > > manually anyway. > > > > > > That said, plus one for deadline_time. Unless there's some other > > modifier > > > better than "deadline_". Don't love the "_at" naming. > > > > > > On Mon, May 26, 2025 at 7:47 PM Kalyan Reddy <kaly...@apache.org> wrote: > > > > > > > Thanks Jens. In light of this, I'm proposing to rename it to > > > > "deadline_at" . I've updated the PR as well. But "deadline_time" also > > > > sounds good. We can go with that as well. I can update PR with what we > > > > agree here. > > > > > > > > On 2025/05/27 01:47:57 Wei Lee wrote: > > > > > Not sure whether "deadline.deadline_time" is an ok idea 🤔 I also > > worry > > > > about what Jens said. > > > > > > > > > > Best, > > > > > Wei > > > > > > > > > > > On May 27, 2025, at 4:26 AM, Jens Scheffler > > > <j_scheff...@gmx.de.INVALID> > > > > wrote: > > > > > > > > > > > > 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 > > > > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > > To unsubscribe, e-mail: dev-unsubscr...@airflow.apache.org > > > > > For additional commands, e-mail: dev-h...@airflow.apache.org > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > To unsubscribe, e-mail: dev-unsubscr...@airflow.apache.org > > > > For additional commands, e-mail: dev-h...@airflow.apache.org > > > > > > > > > > > > > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@airflow.apache.org For additional commands, e-mail: dev-h...@airflow.apache.org
