xBis7 commented on code in PR #43941:
URL: https://github.com/apache/airflow/pull/43941#discussion_r1885697836
##########
airflow/migrations/versions/0017_2_9_2_fix_inconsistency_between_ORM_and_migration_files.py:
##########
@@ -236,6 +236,9 @@ def upgrade():
log_template_id INTEGER,
updated_at TIMESTAMP,
clear_number INTEGER DEFAULT '0' NOT NULL,
+ scheduled_by_job_id INTEGER,
+ context_carrier VARCHAR(50),
+ span_status VARCHAR(50),
Review Comment:
I tried running
```shell
> breeze generate-migration-file -m "add_new_otel_span_fields"
...
...
[2024-12-15T12:56:10.826+0000] {migration.py:207} INFO - Context impl
SQLiteImpl.
[2024-12-15T12:56:10.827+0000] {migration.py:210} INFO - Will assume
non-transactional DDL.
/opt/airflow/airflow/migrations/env.py:133 SAWarning: WARNING: SQL-parsed
foreign key constraint '('dag_id', 'dag', 'dag_id')' could not be located in
PRAGMA foreign_keys for table dag_schedule_asset_alias_reference
No change detected in ORM schema, skipping revision.
```
What am I missing?
--
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]
