Andrey Anshin
Andrey Anshin
> Indeed, this fix the layout, but we lose this nice functionality... If you find a correct way to handle [PEP-657](https://peps.python.org/pep-0657/) in runtime in loggers it would be nice
> Fixed but still raise a warning [tests/serialization/test_pydantic_models.py:162](https://github.com/apache/airflow/blob/15eedd080428314cf6e27443f91624459d17a77d/tests/serialization/test_pydantic_models.py#L162) [tests/serialization/test_pydantic_models.py:163](https://github.com/apache/airflow/blob/15eedd080428314cf6e27443f91624459d17a77d/tests/serialization/test_pydantic_models.py#L163) Could you provide what kind of error you tried to fix?
> Set cascade_backrefs to False in either the relationship() or backref() function for the 2.0 behavior I guess it points to change in models relationships, maybe it is simmilar: https://github.com/sqlalchemy/sqlalchemy/issues/6148
There is one small things, according to the docs it is recommended switch to Future=True after we resolve all RemovedIn20 warnings. https://docs.sqlalchemy.org/en/20/changelog/migration_20.html#migration-to-2-0-step-five-use-the-future-flag-on-session So maybe we need to start replace legacy...
I would rather fix one by one rather than apply fix which might hide this We still have third party components, which strictly depends on SA14 and might not work...
Particular this one described in https://docs.sqlalchemy.org/en/14/orm/cascades.html#save-update And there is description about [`cascade_backrefs`](https://docs.sqlalchemy.org/en/14/orm/relationship_api.html#sqlalchemy.orm.relationship.params.cascade_backrefs) which is `True` by default in 1.4, and `False` in 2.0
> return [{"value": "{{ foo }}"}] return [{"value": "{{ 'foo' }}"}]
Oh... you mean it should be remaining the same. That is interesting behaviour TBH
> interesting . I had no idea pytest is doing that. I've found myself it about week ago when try to find how to solve one issue.
🤔 🤔 🤔 ```console # Provide existent file path, # so loaded YAML file content should be used to call deserialize_model_dict(), rather than None. path = sys.path[0] + "/tests/providers/cncf/kubernetes/pod.yaml" >...