Signal/Android
1
0
Fork 0
mirror of https://github.com/signalapp/Signal-Android.git synced 2025-12-22 03:58:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add stopgap for FTS migration crash.

Browse Source
This commit is contained in:
Greyson Parrelli
2024-08-26 15:58:19 -04:00
parent 60ad879cac
commit 2bc07e87d8
1 changed files with 15 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
app/src/main/java/org/thoughtcrime/securesms/database/helpers/migration/V239_MessageFullTextSearchEmojiSupport.kt
View File

@@ -7,6 +7,7 @@ package org.thoughtcrime.securesms.database.helpers.migration
import android.app.Application import android.app.Application
import net.zetetic.database.sqlcipher.SQLiteDatabase import net.zetetic.database.sqlcipher.SQLiteDatabase
import org.signal.core.util.logging.Log
/** /**
* Recreates the message FTS stuff, but with a tokenizer property that lets us search for emoji. * Recreates the message FTS stuff, but with a tokenizer property that lets us search for emoji.
@@ -15,9 +16,22 @@ import net.zetetic.database.sqlcipher.SQLiteDatabase
@Suppress("ClassName") @Suppress("ClassName")
object V239_MessageFullTextSearchEmojiSupport : SignalDatabaseMigration { object V239_MessageFullTextSearchEmojiSupport : SignalDatabaseMigration {
const val FTS_TABLE_NAME = "message_fts" private val TAG = Log.tag(V239_MessageFullTextSearchEmojiSupport::class.java)
private const val FTS_TABLE_NAME = "message_fts"
override fun migrate(context: Application, db: SQLiteDatabase, oldVersion: Int, newVersion: Int) { override fun migrate(context: Application, db: SQLiteDatabase, oldVersion: Int, newVersion: Int) {
try {
doMigration(db)
} catch (t: Throwable) {
// For some unknown reason, a select few users are hitting a 'vtable constructor' crash when trying to drop the FTS table.
// This feature is not critical, so for now we'll take the loss in those cases and try to synchronize it later.
// The migration doesn't change the actual schema, just the tokenization, so it shouldn't be that big of a deal.
Log.e(TAG, "Failed to perform migration!", t)
}
}
private fun doMigration(db: SQLiteDatabase) {
db.execSQL("DROP TABLE IF EXISTS $FTS_TABLE_NAME") db.execSQL("DROP TABLE IF EXISTS $FTS_TABLE_NAME")
db.execSQL("DROP TABLE IF EXISTS ${FTS_TABLE_NAME}_config") db.execSQL("DROP TABLE IF EXISTS ${FTS_TABLE_NAME}_config")
db.execSQL("DROP TABLE IF EXISTS ${FTS_TABLE_NAME}_content") db.execSQL("DROP TABLE IF EXISTS ${FTS_TABLE_NAME}_content")