Opened 15 years ago

Closed 14 years ago

Last modified 11 years ago

#2798 closed defect (fixed)

Remove ESRI style spatial indexes from Shapefile on update via OGR

Reported by: marisn Owned by: chaitanya
Priority: normal Milestone:
Component: OGR_SF Version: 1.6.0
Severity: normal Keywords: shape
Cc: warmerdam


As OGR doesn't support ESRI style spatial indexes, they get outdated after shapefile editing with OGR. Still ESRI products will use outdated indexes for displaying features (edits done via OGR are not visible till indexes are updated!)

Temporary solution till OGR gets ESRI style spatial indexes for Shapefiles would be to remove them when Shapefile gets updated via OGR.

Change History (6)

comment:1 by warmerdam, 15 years ago

Cc: warmerdam added
Component: defaultOGR_SF
Keywords: shape added
Milestone: 1.6.1
Owner: changed from warmerdam to chaitanya

There are actually two issues.

The first is that we really ought to delete the .sbn and .sbx files (the ESRI style spatial index) whenever the first change is made to shapefile set.

The other is that we should either remove and disable use of the .qix file when we change the file, or work to keep it updated. For now I think we should just delete it as ongoing update will be fairly challenging.

Turning over to Chaitanya to resolve.

comment:2 by chaitanya, 15 years ago

Status: newassigned

comment:3 by chaitanya, 14 years ago

Resolution: fixed
Status: assignedclosed

Added a quick fix for the problem and an extensive test in trunk (r20122). Propagated the fix to 1.7 (r20123) and 1.6 (r20124) branches.

comment:4 by Even Rouault, 14 years ago

Resolution: fixed
Status: closedreopened

ogr_shape_40 fails for me. It looks like you have forgotten to svn add data/gjpoint.sbx data/gjpoint.sbn and data/gjpoint.qix

in reply to:  4 comment:5 by chaitanya, 14 years ago

Resolution: fixed
Status: reopenedclosed

Added the files in r20149.

comment:6 by Even Rouault, 11 years ago

Milestone: 1.6.4

Milestone 1.6.4 deleted

Note: See TracTickets for help on using tickets.