You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
At present, if you cache a query, then the SQL only for that query changes in a subsequent release, you may get an incorrect result unless you clear the cache first, because generated SQL is not included in the id hash. An obvious solution would be to add a version field per query, although this relies on people remembering to bump the version!
The text was updated successfully, but these errors were encountered:
Reflecting on this after getting bitten by a cache version mismatch, I think maybe better to just include the version of fm in the hash. Less error prone at the expense of being more wasteful, but I think that's an acceptable tradeoff.
Maybe one option here would be to append the version to the hash unhashed? That way one could split it out in tests.. You'd need to change the separator to make it a valid table name, or change the table name to not be strictly the same as x<query_id> I guess.
At present, if you cache a query, then the SQL only for that query changes in a subsequent release, you may get an incorrect result unless you clear the cache first, because generated SQL is not included in the id hash. An obvious solution would be to add a version field per query, although this relies on people remembering to bump the version!
The text was updated successfully, but these errors were encountered: