-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
-dialect OGRSQL: COUNT does not validate the column name #9972
Comments
Maybe I would describe the problem somehow like "COUNT does not validate the column name". |
rouault
changed the title
-dialect OGRSQL: count() masks errors
-dialect OGRSQL: COUNT does not validate the column name
May 21, 2024
rouault
added a commit
to rouault/gdal
that referenced
this issue
May 22, 2024
and error out if it doesn't exist, otherwise it evaluates as COUNT(*) Fixes OSGeo#9972
rouault
added a commit
that referenced
this issue
May 28, 2024
and error out if it doesn't exist, otherwise it evaluates as COUNT(*) Fixes #9972
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
With
-dialect SQLite
problems are caught right away,But with
-dialect OGRSQL
,count()
masks the problem,Only after we strip off the
count()
is the problem finally caught,GDAL 3.8.5
The text was updated successfully, but these errors were encountered: