SamWheating commented on issue #37597: URL: https://github.com/apache/airflow/issues/37597#issuecomment-1964943535
> 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 ? -- 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
