Navigation Menu

Skip to content

Commit

Permalink
GPKG: do not run vacuum after raster deletion
Browse files Browse the repository at this point in the history
See my previous commit, for consistency
we do not run VACUUM automatically when
a raster is deleted.

A menu item allows the user to run VACUUM
manually

A possible UX improvement would be to add
a button to the success dialog that
offers the user the option to run VACUUM
after a successful layer deletion.
  • Loading branch information
elpaso committed Sep 21, 2018
1 parent b51cb21 commit 6609e65
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions src/providers/ogr/qgsgeopackagedataitems.cpp
Expand Up @@ -438,16 +438,6 @@ bool QgsGeoPackageCollectionItem::deleteGeoPackageRasterLayer( const QString &ur
);
sqlite3_free( sql );
}
// Vacuum
{
( void )sqlite3_exec(
database.get(), /* An open database */
"VACUUM", /* SQL to be evaluated */
nullptr, /* Callback function */
nullptr, /* 1st argument to callback */
nullptr /* Error msg written here */
);
}

if ( status == SQLITE_OK )
{
Expand Down

0 comments on commit 6609e65

Please sign in to comment.