You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2023/09/04 16:16:16 UTC

[GitHub] [airflow] potiuk commented on pull request #34081: Bring back Pydantic 1 compatibility

potiuk commented on PR #34081:
URL: https://github.com/apache/airflow/pull/34081#issuecomment-1705502737

   > Should we update here too:
   > 
   > https://github.com/apache/airflow/blob/47f79b9198f3350951dc21808c36f889bee0cd06/airflow/serialization/serde.py#L322
   > 
   > ?
   > Like reverting the change in 33956?
   
   Good point. We do not reallly use it (yet)  - it's been added for AIP-44 but it'sa good point to add  check that is v1 and v2 compatible for now 


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org