GPKG: do not run vacuum after raster deletion

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.
qgis · Sep 21, 2018 · 6609e65 · 6609e65
1 parent b51cb21
commit 6609e65
Showing 1 changed file with 0 additions and 10 deletions.
diff --git a/src/providers/ogr/qgsgeopackagedataitems.cpp b/src/providers/ogr/qgsgeopackagedataitems.cpp
@@ -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 )
         {