amoghrajesh commented on PR #59711: URL: https://github.com/apache/airflow/pull/59711#issuecomment-3688919197
> > 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. I do not plan to handle the record migration using DB migrations -- that doesn't seem to be clean to me and could be expensive to do at migration stage too. I have maintained compat instead at the application layer. -- 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]
