Makussu/subtracks
1
0
Fork 0
mirror of https://github.com/austinried/subtracks.git synced 2026-11-11 15:29:15 +01:00
Code Issues Packages Projects Releases Wiki Activity

logging: drop the noisy album count from the sync summary

Browse Source 
sum(artist.albumCount) double-counts compilation albums, so it read higher than the albums actually synced; keep only the song total, which matches exactly.
This commit is contained in:
austinried
2026-10-09 07:58:55 +00:00
parent d4152ba3a1
commit a8d6111aaa
2 changed files with 1 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/main/kotlin/com/subtracks/data/sync/SyncManager.kt
+1 -1
View File
@@ -69,7 +69,7 @@ class SyncManager(
Log.i( Log.i(
"sync", "sync",
"finished in ${elapsedMs(startedAt)}ms: " + "finished in ${elapsedMs(startedAt)}ms: " +
"${summary.artists} artists, ${summary.albums} albums (server reported ${summary.reportedAlbums}), " + "${summary.artists} artists, ${summary.albums} albums, " +
"${summary.songs} songs (server reported ${summary.reportedSongs}), " + "${summary.songs} songs (server reported ${summary.reportedSongs}), " +
"${summary.playlists} playlists, ${summary.playlistSongs} playlist tracks, ${summary.pruned} pruned", "${summary.playlists} playlists, ${summary.playlistSongs} playlist tracks, ${summary.pruned} pruned",
) )
app/src/main/kotlin/com/subtracks/data/sync/SyncService.kt
-4
View File
@@ -26,7 +26,6 @@ class SyncService(
private var playlistsSynced = 0 private var playlistsSynced = 0
private var playlistSongsSynced = 0 private var playlistSongsSynced = 0
private var pruned = 0 private var pruned = 0
private var reportedAlbums = 0L
private var reportedSongs = 0L private var reportedSongs = 0L
suspend fun sync(): SyncSummary { suspend fun sync(): SyncSummary {
@@ -43,7 +42,6 @@ class SyncService(
playlists = playlistsSynced, playlists = playlistsSynced,
playlistSongs = playlistSongsSynced, playlistSongs = playlistSongsSynced,
pruned = pruned, pruned = pruned,
reportedAlbums = reportedAlbums,
reportedSongs = reportedSongs, reportedSongs = reportedSongs,
) )
} }
@@ -56,7 +54,6 @@ class SyncService(
write("artists", artistColumns, primaryKey, batch) { it.values() } write("artists", artistColumns, primaryKey, batch) { it.values() }
batch.forEach { seen.add(idHash(it.id)) } batch.forEach { seen.add(idHash(it.id)) }
artistsSynced += batch.size artistsSynced += batch.size
reportedAlbums += batch.sumOf { it.albumCount }
} }
pruned += pruneStaleIds(seen, { library.artistIdsAfter(source.id, it, PRUNE_PAGE) }, { library.deleteArtists(source.id, it) }) pruned += pruneStaleIds(seen, { library.artistIdsAfter(source.id, it, PRUNE_PAGE) }, { library.deleteArtists(source.id, it) })
} }
@@ -244,7 +241,6 @@ data class SyncSummary(
val playlists: Int, val playlists: Int,
val playlistSongs: Int, val playlistSongs: Int,
val pruned: Int, val pruned: Int,
val reportedAlbums: Long,
val reportedSongs: Long, val reportedSongs: Long,
) )