Bug 385834 don't do incremental vacuuming, makes fragmentation worse [was: places' sqlite file can get overly large, do incremental vacuuming] (r=sspitzer)

This commit is contained in:
dietrich@mozilla.com 2007-11-02 10:46:41 -07:00
parent d717255ca2
commit c4d846ebec
2 changed files with 25 additions and 50 deletions

View File

@ -151,10 +151,12 @@
#endif // LAZY_ADD
// check idle timer every 5 minutes
#define IDLE_TIMER_TIMEOUT (300 * PR_MSEC_PER_SEC)
// perform vacuum every 15 mins *** CURRENTLY DISABLED ***
// 15 minutes = 900 seconds = 900000 milliseconds
#define VACUUM_IDLE_TIME_IN_MSECS (900000)
// check every 5 minutes
#define VACUUM_TIMER_TIMEOUT (300 * PR_MSEC_PER_SEC)
NS_IMPL_ADDREF(nsNavHistory)
NS_IMPL_RELEASE(nsNavHistory)
@ -520,18 +522,12 @@ nsNavHistory::InitDB(PRBool *aDoImport)
rv = mDBConn->ExecuteSimpleSQL(pageSizePragma);
NS_ENSURE_SUCCESS(rv, rv);
// Set the database up for incremental vacuuming.
// if the database was created before we started doing
// incremental vacuuming, this will have no effect.
rv = mDBConn->ExecuteSimpleSQL(NS_LITERAL_CSTRING("PRAGMA auto_vacuum=2"));
if (!mIdleTimer) {
mIdleTimer = do_CreateInstance("@mozilla.org/timer;1", &rv);
NS_ENSURE_SUCCESS(rv, rv);
if (!mVacuumTimer) {
mVacuumTimer = do_CreateInstance("@mozilla.org/timer;1", &rv);
NS_ENSURE_SUCCESS(rv, rv);
rv = mVacuumTimer->InitWithFuncCallback(VacuumTimerCallback, this,
VACUUM_TIMER_TIMEOUT,
rv = mIdleTimer->InitWithFuncCallback(IdleTimerCallback, this,
IDLE_TIMER_TIMEOUT,
nsITimer::TYPE_REPEATING_SLACK);
NS_ENSURE_SUCCESS(rv, rv);
}
@ -3401,7 +3397,7 @@ nsNavHistory::AddDocumentRedirect(nsIChannel *aOldChannel,
}
nsresult
nsNavHistory::PerformVacuumIfIdle()
nsNavHistory::OnIdle()
{
nsresult rv;
nsCOMPtr<nsIIdleService> idleService =
@ -3412,29 +3408,9 @@ nsNavHistory::PerformVacuumIfIdle()
rv = idleService->GetIdleTime(&idleTime);
NS_ENSURE_SUCCESS(rv, rv);
// if we've been idle for more than VACUUM_IDLE_TIME_IN_MSECS
// incrementally vacuum
// If we've been idle for more than VACUUM_IDLE_TIME_IN_MSECS
// perform a vacuum.
if (idleTime > VACUUM_IDLE_TIME_IN_MSECS) {
PRInt32 vacuum;
nsCOMPtr<mozIStorageStatement> statement;
rv = mDBConn->CreateStatement(NS_LITERAL_CSTRING("PRAGMA auto_vacuum"),
getter_AddRefs(statement));
NS_ENSURE_SUCCESS(rv, rv);
PRBool hasResult;
rv = statement->ExecuteStep(&hasResult);
NS_ENSURE_SUCCESS(rv, rv);
NS_ENSURE_TRUE(hasResult, NS_ERROR_FAILURE);
vacuum = statement->AsInt32(0);
// if our database was created with incremental_vacuum,
// do incremental vacuuming
if (vacuum == 2) {
rv = mDBConn->ExecuteSimpleSQL(
NS_LITERAL_CSTRING("PRAGMA incremental_vacuum;"));
NS_ENSURE_SUCCESS(rv, rv);
}
else {
#if 0
// Currently commented out because compression is very slow
// see bug #390244 for more details
@ -3444,15 +3420,14 @@ nsNavHistory::PerformVacuumIfIdle()
NS_ENSURE_SUCCESS(rv, rv);
#endif
}
}
return NS_OK;
}
void // static
nsNavHistory::VacuumTimerCallback(nsITimer* aTimer, void* aClosure)
nsNavHistory::IdleTimerCallback(nsITimer* aTimer, void* aClosure)
{
nsNavHistory* history = static_cast<nsNavHistory*>(aClosure);
(void)history->PerformVacuumIfIdle();
(void)history->OnIdle();
}
// nsIObserver *****************************************************************
@ -3462,9 +3437,9 @@ nsNavHistory::Observe(nsISupports *aSubject, const char *aTopic,
const PRUnichar *aData)
{
if (nsCRT::strcmp(aTopic, gQuitApplicationMessage) == 0) {
if (mVacuumTimer) {
mVacuumTimer->Cancel();
mVacuumTimer = nsnull;
if (mIdleTimer) {
mIdleTimer->Cancel();
mIdleTimer = nsnull;
}
if (mAutoCompleteTimer) {
mAutoCompleteTimer->Cancel();

View File

@ -610,9 +610,9 @@ protected:
// updating during import.
nsresult CreateLookupIndexes();
nsresult PerformVacuumIfIdle();
nsCOMPtr<nsITimer> mVacuumTimer;
static void VacuumTimerCallback(nsITimer* aTimer, void* aClosure);
nsCOMPtr<nsITimer> mIdleTimer;
static void IdleTimerCallback(nsITimer* aTimer, void* aClosure);
nsresult OnIdle();
PRInt64 mTagRoot;
PRInt64 GetTagRoot();