Conversation
Pull Request Test Coverage Report for Build 797
💛 - Coveralls |
|
@abondar This PR has two basic things:
I am going to split this up into the tests & bugfixes (backport for 0.13.6). But please have a look at this PR :-) |
|
Looks all good to me |
|
Yes, the speedup was mainly on two "special" cases:
This gave a better speedup with larger models with more fields. We could probably do a similar speedup for creating from Python via the constructor, and the converting to the DB fields. But we are running out of Python-level inefficiencies to optimize. We'll soon have to expand the benchmarking system to test DB/connector/Tortoise interactions. e.g. Are we using the indexes correctly? Is there any unnecessary overhead with the DB connectors for MySQL/PostgreSQL? Do we set up the DB fields correctly for case-insensitive operations? etc... |
The plan is to have this as the next master base. I'm leaving this here as a PR so its easier to manage.
DONE:
flyntandpyupgradeto help automate)Model._init_from_dbto minimize callingto_python_valueif we have guarantees that allow for it to be skipped.