Opened 8 years ago

Closed 5 years ago

Last modified 5 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 6 years ago.
First phase at supporting irregular tile grids.
irregular_tile_support_phase2.patch (29.0 KB) - added by dzwarg 5 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 5 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 Changed 8 years ago by jorgearevalo

Status: newassigned

comment:2 Changed 7 years ago by jorgearevalo

Summary: WKT Raster driver: allow non-regular blocked rastersPostGIS Raster driver: allow non-regular blocked rasters

comment:3 Changed 7 years ago by jorgearevalo

Keywords: postgis raster added; wkt removed
Milestone: 1.7.01.8.0

comment:4 Changed 6 years ago by jorgearevalo

Milestone: 1.8.11.8.2

comment:5 Changed 6 years ago by jorgearevalo

Priority: normalhighest

comment:6 Changed 6 years ago by jorgearevalo

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

comment:7 Changed 6 years ago by warmerdam

Milestone: 1.8.2

removing milestone.

comment:8 Changed 6 years ago by pracine

Keywords: postgis_raster added; postgis raster removed

comment:9 Changed 6 years ago by jorgearevalo

Milestone: 1.9.1

comment:10 Changed 6 years ago by Mateusz Łoskot

Keywords: postgisraster wktraster added

Changed 6 years ago by dzwarg

First phase at supporting irregular tile grids.

comment:11 Changed 6 years ago by dzwarg

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 Changed 5 years ago by dzwarg

This patch also resolves the buffer overflow described in #4703

comment:13 Changed 5 years ago by Mateusz Łoskot

Cc: Mateusz Łoskot added

Changed 5 years ago by dzwarg

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

Changed 5 years ago by dzwarg

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 Changed 5 years ago by Even Rouault

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 Changed 5 years ago by dzwarg

Resolution: fixed
Status: assignedclosed

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

comment:16 Changed 5 years ago by dzwarg

Fixed in r24675, btw.

Note: See TracTickets for help on using tickets.