New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug] temporal tools fail with SQLite 3.31 #3466
Comments
Ops, sorry! I think the culprit is actually: #3359 3.33 is available in:
So, Ubuntu 20.04 ships 3.31, which i not compatible... 20.04 is End of Standardsupport by April 2025, so ~next year... What should we do? Revert #3359 until then? #3350 is the bigger performance improvement and should not increase the required SQLite verion.... |
You could perhaps do a version check, like: grass/scripts/db.dropcolumn/db.dropcolumn.py Lines 97 to 106 in 76bcffb
and use the syntax supported for older versions only there (even if it is slower). |
Are the SQLite operation done by Python? In that case, is the version of SQLite dependent on the Python version (if from the standard library, ie not using shared library)? |
This needs to be resolved (one way or the other) before the 8.4 release. What do you think @ninsbl? |
Hm, I would suggest to revert #3359, re-open the PR and wait with merging until Ubuntu 20.04 is out of support (which is less than a year from now)... |
Is there a way that both can be kept until then? What amount of performance regression would we be trading with a revert like that? (We would be worsening the goal of #2185 no?) |
We could reactivate the old templates and use them depending on the DB backend and version. Maybe I can look into that tonight... |
Great! Still one year is quite long, and I'd really like that the 8.4.0 release would be out before the community sprint, so we could be ambitious with our development during that week, instead of trying to catch up with the release that is due, or still taking some time to test out the RCs. @wenzeslaus or @neteler, is it still realistic to be finished with 8.4.0 before the sprint or I'm only dreaming? |
In our recent PSC meeting we decided that the release should happen before the community meeting! |
Describe the bug
The temporal tools fail with SQLite 3.31 although minimal supported version according to #3350 is 3.30 (REQUIREMENTS.md does not specify a minimal SQLite version).
5e3858b works and 0c73ced does not, so this points to #3350.
To Reproduce
Steps to reproduce the behavior:
Run test:
Possibly like this:
Expected behavior
According to #3350, the code should work with SQLite 3.30, but it does not work for me with 3.31. Maybe the requirements are higher?
Screenshots
System description
Additional context
CI works with Ubuntu 22.04.
Clearly, I should upgrade from 20.04, but that's not the point here :-)
The text was updated successfully, but these errors were encountered: