Skip to content

Commit 8dc9f88

Browse files
Add executor for MSP Room transactions (#5901)
Task/Issue URL: https://app.asana.com/1/137249556945/task/1209959841243986?focus=true ### Description Use dedicated executor for MSP transactions to prevent thread exhaustion on Room executors ### Steps to test this PR _Feature 1_ - [ ] - [ ] ### UI changes | Before | After | | ------ | ----- | !(Upload before screenshot)|(Upload after screenshot)| --------- Co-authored-by: Cristian Monforte <[email protected]>
1 parent 6bf0bad commit 8dc9f88

File tree

1 file changed

+3
-0
lines changed
  • malicious-site-protection/malicious-site-protection-impl/src/main/kotlin/com/duckduckgo/malicioussiteprotection/impl

1 file changed

+3
-0
lines changed

malicious-site-protection/malicious-site-protection-impl/src/main/kotlin/com/duckduckgo/malicioussiteprotection/impl/MaliciousSiteModule.kt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ import dagger.Provides
3232
import dagger.SingleInstanceIn
3333
import java.io.File
3434
import java.security.MessageDigest
35+
import java.util.concurrent.Executors
3536
import java.util.concurrent.TimeUnit.SECONDS
3637
import javax.inject.Named
3738
import javax.inject.Qualifier
@@ -56,6 +57,8 @@ class MaliciousSiteModule {
5657
fun provideMaliciousSiteProtectionDatabase(context: Context): MaliciousSitesDatabase {
5758
return Room.databaseBuilder(context, MaliciousSitesDatabase::class.java, "malicious_sites.db")
5859
.addMigrations(*ALL_MIGRATIONS)
60+
.setQueryExecutor(Executors.newFixedThreadPool(4))
61+
.setTransactionExecutor(Executors.newSingleThreadExecutor())
5962
.fallbackToDestructiveMigration()
6063
.build()
6164
}

0 commit comments

Comments
 (0)