Opened 9 years ago
Closed 8 years ago
#3396 closed defect (fixed)
[regression] ST_EstimatedExtent errors out when no stats are found
Reported by: | strk | Owned by: | pramsey |
---|---|---|---|
Priority: | high | Milestone: | PostGIS 2.3.0 |
Component: | postgis | Version: | master |
Keywords: | Cc: |
Description
According to #877 this was fixed 4 years ago, and the PostGIS Manual indeed reports it should return NULL instead since 1.5.4 — http://postgis.net/docs/manual-2.2/ST_EstimatedExtent.html
In current trunk, I see it raising an exception instead. It's to be checked WHEN it regressed, and I think we should make it match the documentation…
Change History (8)
comment:1 by , 9 years ago
comment:2 by , 9 years ago
POSTGIS="2.0.8SVN r14008" returns NULL.
Quick testcase:
CREATE TABLE t(g geometry); SELECT ST_Estimated_Extent('t','g');
comment:3 by , 9 years ago
comment:4 by , 9 years ago
So it looks like 2.1.0 was the first to switch back to ERROR. What do we do now ? Current situation:
1.0.0 - 1.5.3 : ERROR 1.5.4 - 1.5.10 : WARNING 2.0.0 - 2.0.8dev : WARNING 2.1.0 - 2.1.9dev : ERROR 2.2.0 - 2.2.1dev : ERROR
comment:5 by , 9 years ago
Going back to warning shouldn't be a problem, it's unlikely anyone depends on error. However, apparently it's sufficiently rare that hardly anyone depends on warning either
comment:6 by , 8 years ago
Milestone: | PostGIS 2.3.0 → PostGIS 2.2.3 |
---|
comment:7 by , 8 years ago
Milestone: | PostGIS 2.2.3 → PostGIS 2.3.0 |
---|
I've decided shouldn't be done in micro, so will do in 2.3.0
The commit that switched from ERROR to WARNING for 1.5.4: https://trac.osgeo.org/postgis/changeset/8319