kaxil commented on PR #59711:
URL: https://github.com/apache/airflow/pull/59711#issuecomment-3688282049

   >Due to above, ExtendedJSON (which auto-serializes) is swapped with plain 
JSON/JSONB columns. This removes the automatic serialization at the database 
layer.
   
   `ExtendedJSON` was already using `JSON`/`JSONB` under the hood. 
   
   
https://github.com/apache/airflow/blob/074548d76d7c2e2579dc0d603e4503eb7b58752f/airflow-core/src/airflow/utils/sqlalchemy.py#L123-L139
   
   So it isn't a DB column change though so we don't need DB migration unless 
you plan to handle the existing records to new format.
   
   
   `DagRun.context_carrier` also does the same btw -- we should change that too


-- 
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]

Reply via email to