Opened 12 years ago
Last modified 12 years ago
#4416 closed defect
PostGIS raster driver — at Initial Version
Reported by: | robe | Owned by: | warmerdam |
---|---|---|---|
Priority: | normal | Milestone: | 1.9.1 |
Component: | GDAL_Raster | Version: | svn-trunk |
Severity: | normal | Keywords: | postgis_raster, postgisraster, wktraster |
Cc: | jorgearevalo, dzwarg |
Description
I guess its probably too late to get these fixes in for 1.9.0 since youa re at rc1 already.
Couple of issues with PostGIS raster driver detailed in http://trac.osgeo.org/postgis/ticket/1422
postgisraserrasterband.cpp is using obsolete column names.
Line 118:
osCommand.Printf("select o_table_name, overview_factor, o_column, " "o_table_schema from raster_overviews where r_table_schema = " "'%s' and r_table_name = '%s' and r_column = '%s'", poDS->pszSchema, poDS->pszTable, poDS->pszColumn);
should be:
osCommand.Printf("select o_table_name, overview_factor, o_raster_column, " "o_table_schema from raster_overviews where r_table_schema = " "'%s' and r_table_name = '%s' and r_column = '%s'", poDS->pszSchema, poDS->pszTable, poDS->pszColumn);
New raster meta views are documented in: http://www.postgis.org/documentation/manual-svn/using_raster.xml.html#RT_Raster_Overviews
and http://www.postgis.org/documentation/manual-svn/using_raster.xml.html#RT_Raster_Columns
Also the use of construct:
st_setsrid(st_makebox2d(st_point(%f, %f), st_point(%f," "%f)),%d)
shoudl be replaced with:
st_makeenvelope(%f, %f, %f,%f,%d)
As documented here: http://www.postgis.org/documentation/manual-svn/ST_MakeEnvelope.html
Since we are slowly phasing out the use of box2d