filippodb
/
SchildiChat-android
mirror of https://github.com/SchildiChat/SchildiChat-android.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Disable timeline cleanup mechanisms that are no longer necessary 

Element's new timeline alg should keep is free from timeline loops and
empty chunks.
Thus, removing these checks should give us back a little bit of faster
room loading.

Change-Id: I7c6bc314b98410dd47af44e958422860f26771ce
This commit is contained in:
SpiritCroc 2022-09-17 17:23:46 +02:00
parent 991a658221
commit 7d4c489816
1 changed files with 4 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/room/timeline/LoadTimelineStrategy.kt
View File

@ -60,14 +60,10 @@ import java.util.concurrent.atomic.AtomicReference
* Once we got a ChunkEntity we wrap it with TimelineChunk class so we dispatch any methods for loading data.
*/
// Whether to search for timeline loops, and fix them.
// TODO: once we feel comfortable that this is no longer necessary,
// we probably want to disable this again for improving performance.
const val ENABLE_TIMELINE_LOOP_SPLITTING = true
// Whether to search for stuck timelines due to empty self-linking chunks.
// TODO: once we feel comfortable that this is no longer necessary,
// we probably want to disable this again for improving performance.
const val ENABLE_TIMELINE_EMPTY_CHUNK_CLEANUP = true
// [No longer necessary] Whether to search for timeline loops, and fix them.
const val ENABLE_TIMELINE_LOOP_SPLITTING = false
// [No longer necessary] Whether to search for stuck timelines due to empty self-linking chunks.
const val ENABLE_TIMELINE_EMPTY_CHUNK_CLEANUP = false
// Performance consideration for huge timelines, when having ENABLE_TIMELINE_LOOP_SPLITTING or ENABLE_TIMELINE_EMPTY_CHUNK_CLEANUP set to true
const val MAX_CHUNK_REPAIR_CHECK_COUNT = 100