Opened 14 years ago

Closed 12 years ago

Last modified 12 years ago

#3233 closed task (fixed)

PostGIS Raster driver: allow non-regular blocked rasters

Reported by: jorgearevalo Owned by: jorgearevalo
Priority: highest Milestone: 1.9.1
Component: GDAL_Raster Version: svn-trunk
Severity: normal Keywords: postgis_raster, postgisraster, wktraster
Cc: dzwarg, Mateusz Łoskot

Description

The driver at its current state only allows regular blocked rasters.

Attachments (3)

irregular_tile_support_phase1.patch​ (26.3 KB ) - added by dzwarg 12 years ago.
First phase at supporting irregular tile grids.
irregular_tile_support_phase2.patch​ (29.0 KB ) - added by dzwarg 12 years ago.
More performant method which checks for primary/unique key only when necessary.
irregular_tile_support_phase3.patch​ (37.2 KB ) - added by dzwarg 12 years ago.
More performant method which checks for primary/unique key only when necessary + removal of all ST_Metadata calls & consolidation of SetRasterProperties and SetRasterBand methods. Warning: this patch is memory leaky.

Download all attachments as: .zip

Change History (19)

comment:1 by jorgearevalo, 14 years ago

Status: new → assigned

comment:2 by jorgearevalo, 14 years ago

Summary: WKT Raster driver: allow non-regular blocked rasters → PostGIS Raster driver: allow non-regular blocked rasters

comment:3 by jorgearevalo, 14 years ago

Keywords: postgis raster added; wkt removed
Milestone: 1.7.0 → 1.8.0

comment:4 by jorgearevalo, 13 years ago

Milestone: 1.8.1 → 1.8.2

comment:5 by jorgearevalo, 13 years ago

Priority: normal → highest

comment:6 by jorgearevalo, 13 years ago

Working on it, with the implemented IRasterIO function (r23098). I have the guidelines

comment:7 by warmerdam, 12 years ago

Milestone: 1.8.2

removing milestone.

comment:8 by pracine, 12 years ago

Keywords: postgis_raster added; postgis raster removed

comment:9 by jorgearevalo, 12 years ago

Milestone: → 1.9.1

comment:10 by Mateusz Łoskot, 12 years ago

Keywords: postgisraster wktraster added

by dzwarg, 12 years ago

Attachment: irregular_tile_support_phase1.patch​ added

First phase at supporting irregular tile grids.

comment:11 by dzwarg, 12 years ago

Cc: dzwarg added

Created patch that begins to incorporate the architecture described in the ​GDALDriverSpecificationworking wiki page.

Specifically:

  • Remove the checks for the 'geometry' datatype
  • Remove the checks for the spatial tables
  • Remove the metadata query that retrieves the metadata for all sub-dataset rasters

comment:12 by dzwarg, 12 years ago

This patch also resolves the buffer overflow described in #4703

comment:13 by Mateusz Łoskot, 12 years ago

Cc: Mateusz Łoskot added

by dzwarg, 12 years ago

Attachment: irregular_tile_support_phase2.patch​ added

More performant method which checks for primary/unique key only when necessary.

by dzwarg, 12 years ago

Attachment: irregular_tile_support_phase3.patch​ added

More performant method which checks for primary/unique key only when necessary + removal of all ST_Metadata calls & consolidation of SetRasterProperties and SetRasterBand methods. Warning: this patch is memory leaky.

comment:14 by Even Rouault, 12 years ago

Changeset r24664 by jorgearevalo: Applied patch 3 from ticket #3233 . Does this close the ticket ? (the milesone should be updated, if it is not backported)

comment:15 by dzwarg, 12 years ago

Resolution: → fixed
Status: assigned → closed

Fixed memory leaks from patch 3. Irregular grids now supported in the PostGIS Raster GDAL driver.

comment:16 by dzwarg, 12 years ago

Fixed in r24675, btw.

Note: See TracTickets for help on using tickets.