bbovenzi commented on issue #37597: URL: https://github.com/apache/airflow/issues/37597#issuecomment-1965244838
> > But doing this fully could require a big migration of the audit logs table. > > I guess that depends, should we add the empty column and then start populating it moving forwards, or should we aim to repopulate the run_id for previous Log rows based on a join with the DagRun table (similar to [this](https://github.com/apache/airflow/blob/54790274c2edf39c06f494a900bc85523349a77f/airflow/migrations/versions/0102_2_3_0_switch_xcom_table_to_use_run_id.py)) > > Not really sure what the standard is here, and this migration might be pretty hefty due to the potentially high volume of the Log table. > > Thoughts, @bbovenzi ? I'm not opposed to only populating it moving forward. Going through old logs and translating execution_date to dag_id will be a heavy lift. Already we do not record `execution_date` often enough so many logs are already lacking information. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
