Flip fixed-format cache to on by default#20758
Conversation
|
According to mypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅ |
JukkaL
left a comment
There was a problem hiding this comment.
Let's do it! Maybe we should also enable sqlite cache by default (in 1.20), for even more incremental mode performance gains?
Maybe. I am thinking maybe we should also add some helper script to extract cached content from DB, or do we already have one? |
|
Btw, I just noticed that some of the scripts (like |
We already have it:
These are all worth fixing. I think they mostly work at file level, so supporting fixed format caches shouldn't be difficult, but I haven't looked into this in detail. Maybe we can just process the binary data for each .ff file as a single entity instead of using deserialized json data. The dependency cache files have special processing, but are they even using the fixed format? |
OK, I will make a PR to fix
Deps files are still JSON, and TBH I remember very little about them, so I would prefer if you take a look at the cache diff scripts. |
|
It turns out fixing |
Since this is scheduled for v1.20 it is better do this sooner than later. So that people who are using mypy master can give some early feedback.