Feature/epg refresh performance optimization#317
Conversation
|
Looks good to me. I tested it against latest main: it merges cleanly, Play/Sideload Kotlin compile passes, and the new IPTV optimization tests pass. Only small cleanup: After that, I think this is safe to merge. Also PR317 includes the same core fixes as PR316 plus extra EPG performance work, so we probably don’t need PR316 separately if this one is merged. |
|
check now
…On Fri, Jun 5, 2026 at 7:04 PM Prodigy ***@***.***> wrote:
*ProdigyV21* left a comment (ProdigyV21/ARVIO#317)
<#317 (comment)>
Looks good to me. I tested it against latest main: it merges cleanly,
Play/Sideload Kotlin compile passes, and the new IPTV optimization tests
pass.
Only small cleanup: git diff --check reports trailing whitespace in
IptvRepository.kt and IptvRepositoryOptimizationTest.kt. This is not a
runtime/app issue, just invisible extra spaces at the end of some lines,
but it can fail style/CI checks and is easy to clean.
After that, I think this is safe to merge. Also PR317 includes the same
core fixes as PR316 plus extra EPG performance work, so we probably don’t
need PR316 separately if this one is merged.
—
Reply to this email directly, view it on GitHub
<#317?email_source=notifications&email_token=BOC4WIBIOLF37FWY3LSQ43D46LD5FA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINRTGIYDEOBSGYYKM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#issuecomment-4632028260>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BOC4WIBSWZCWBMMOQFBU2K346LD5FAVCNFSM6AAAAACZ2UZQICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DMMZSGAZDQMRWGA>
.
Triage notifications, keep track of coding agent tasks and review pull
requests on the go with GitHub Mobile for iOS
<https://github.com/notifications/mobile/ios/BOC4WIG7MCUW2KY6XKKCSML46LD5FA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINRTGIYDEOBSGYYKM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KUZTPN52GK4S7NFXXG>
and Android
<https://github.com/notifications/mobile/android/BOC4WIF7PWE66452YPRGWCL46LD5FA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINRTGIYDEOBSGYYKM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2K4ZTPN52GK4S7MFXGI4TPNFSA>.
Download it today!
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
PR317 now conflicts with latest main because PR316 has already been merged. Please rebase PR317 on latest main and keep the PR316 changes that are already there. Then reapply only the extra PR317 parts: guide-key candidate caching, ETag/Last-Modified/HTTP 304 handling, and incremental EPG refresh persistence changes. Current conflicts are in:
After it merges cleanly again, I can retest compile and unit tests. Same in PR318 you did. |
|
Will do!
…On Fri, 5 Jun, 2026, 7:26 pm Prodigy, ***@***.***> wrote:
*ProdigyV21* left a comment (ProdigyV21/ARVIO#317)
<#317 (comment)>
PR317 now conflicts with latest main because PR316 has already been merged.
Please rebase PR317 on latest main and keep the PR316 changes that are
already there. Then reapply only the extra PR317 parts: guide-key candidate
caching, ETag/Last-Modified/HTTP 304 handling, and incremental EPG refresh
persistence changes.
Current conflicts are in:
- IptvRepository.kt
- IptvRepositoryOptimizationTest.kt
After it merges cleanly again, I can retest compile and unit tests. Same
in PR318 you did.
—
Reply to this email directly, view it on GitHub
<#317?email_source=notifications&email_token=BOC4WICXVHB26CM3GPCKOL346LGSLA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINRTGIZTENRTGMYKM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#issuecomment-4632326330>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BOC4WIFODN6W4OUPNAKQT3T46LGSLAVCNFSM6AAAAACZ2UZQICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DMMZSGMZDMMZTGA>
.
Triage notifications, keep track of coding agent tasks and review pull
requests on the go with GitHub Mobile for iOS
<https://github.com/notifications/mobile/ios/BOC4WIERPZEW4KSMWUQEDWT46LGSLA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINRTGIZTENRTGMYKM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KUZTPN52GK4S7NFXXG>
and Android
<https://github.com/notifications/mobile/android/BOC4WIFKSZ4ER6VKY6FEPM346LGSLA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINRTGIZTENRTGMYKM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2K4ZTPN52GK4S7MFXGI4TPNFSA>.
Download it today!
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
93e3709 to
376d266
Compare
📝 Description
This PR introduces a set of reliability and performance improvements for the IPTV and EPG loading pipeline.
Reliability Improvements
XMLTV Parsing Optimization
Improved
BackslashEscapeSanitizingInputStreamto correctly process bulk reads used by XML parsers.Previously, sanitization logic was only applied to single-byte reads, allowing malformed XMLTV data to bypass sanitization during normal parser execution.
The implementation now performs efficient block-based sanitization while preserving compatibility with existing XMLTV workflows.
Concurrent Playlist Aggregation Safety
Resolved a concurrency issue during simultaneous playlist loading where channel aggregation callbacks could trigger
ConcurrentModificationExceptionwhile playlists were being processed concurrently.Channel snapshots are now generated using synchronized access before UI updates are dispatched.
Performance Improvements
Guide Key Candidate Caching
Added an in-memory cache for guide key candidate generation to reduce repeated normalization and alias generation work during EPG matching.
HTTP Conditional Requests
Added support for:
This allows unchanged EPG sources to reuse existing guide data without reprocessing large XMLTV datasets.
Incremental EPG Refresh
Improved refresh workflows to avoid unnecessary index persistence and guide regeneration when XMLTV content has not changed.
This reduces disk operations and refresh overhead for large IPTV installations.
Test Coverage
Added dedicated tests covering:
Benefits
Testing