commit 3ac8148cdc79bde4b009ccb6a9ffda77bdb547d3
author Andy McFadden <fadden@android.com> Fri Dec 02 16:47:38 2011 -0800
committer Andy McFadden <fadden@android.com> Mon Dec 05 10:42:50 2011 -0800
tree 822be16b6d7cf379e38e71aa81e0d918cf5db3e8
parent 7c5e457794b94b25774949fdc6f2e02e59c8a074

Speed up database update

Drop the trigger on sync_id before doing a bulk update of sync_id
fields.  This dramatically reduces the time required for the
calendar database update when there are lots of events.

Bug 5699796

Change-Id: I94c2fb624b5b63bfdf760b3092697f9205a634c9

src/com/android/providers/calendar/CalendarDatabaseHelper.java

diff --git a/src/com/android/providers/calendar/CalendarDatabaseHelper.java b/src/com/android/providers/calendar/CalendarDatabaseHelper.java
index d609bdf..db70269 100644
--- a/src/com/android/providers/calendar/CalendarDatabaseHelper.java
+++ b/src/com/android/providers/calendar/CalendarDatabaseHelper.java
@@ -1512,7 +1512,11 @@
         * Change event id's from ".../private/full/... to .../events/...
         * Set Calendars.canPartiallyUpdate to 1 to support partial updates
         * Nuke sync state so we re-sync with a fresh etag and edit url
+        *
+        * We need to drop the original_sync_update trigger because it fires whenever the
+        * sync_id field is touched, and dramatically slows this operation.
         */
+        db.execSQL("DROP TRIGGER IF EXISTS original_sync_update");
         db.execSQL("UPDATE Events SET "
                 + "_sync_id = REPLACE(_sync_id, '/private/full/', '/events/'), "
                 + "original_sync_id = REPLACE(original_sync_id, '/private/full/', '/events/') "
@@ -1520,6 +1524,7 @@
                 +    "JOIN Calendars ON Events.calendar_id = Calendars._id "
                 +    "WHERE account_type = 'com.google')"
         );
+        db.execSQL(CREATE_SYNC_ID_UPDATE_TRIGGER);
 
         db.execSQL("UPDATE Calendars SET canPartiallyUpdate = 1 WHERE account_type = 'com.google'");