Upgrade SQLAlchemy to 2.0.2#86436
Conversation
|
Hey there @home-assistant/core, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
|
Converted to draft, until SQLAlchemy 2.0.0 is released |
2d7dbc7 to
a22848d
Compare
|
Hmm.. tests are passing locally |
|
postgresql looks good as well with the caveat that the dataset I'm using and the number of entities isn't as good as the mariadb/sqlite testing since I don't have a production install that uses postgresql, only a test one that has been running for a while with a smaller set of integrations. |
|
Testing is good on this. From my perspective this seems GTG now |
|
unmarking draft since 2.0.0 is out #86436 (comment) |
|
Let's keep this as draft until a week after 2023.2 is released |
|
Sounds like a good idea. There is bound to be something that might conflict 👍 |
|
Something is wrong with logbook queries. They are taking 10x longer than expected |
|
Its not this PR, its #87108 |
|
Will merge tomorrow per previous discussion unless I hear otherwise. |
Proposed change
Upgrade SQLAlchemy to 2.0.2
https://docs.sqlalchemy.org/en/20/changelog/changelog_20.html#change-2.0.2
A workaround added to the tests until a release with
sqlalchemy-utilsincluding kvesteri/sqlalchemy-utils#677 is available.Production testing:
Performance testing
Type of change
Additional information
Checklist
black --fast homeassistant tests)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest.requirements_all.txt.Updated by running
python3 -m script.gen_requirements_all..coveragerc.To help with the load of incoming pull requests: