tree e42136e60650bf9061728a33ba72d3f181bdd19f
parent a1c9557a40e48e248c174c75b03b70e4da6292c6
author Clara Fok <clarafok@google.com> 1697502942 -0700
committer Android Build Coastguard Worker <android-build-coastguard-worker@google.com> 1700540156 +0000

LimitOffsetPagingSource to return LoadResult.Error

Currently any Exception resulting from calling LimitOffsetPagingSource.load isn't caught, resulting in app crashes unless users caught it as part of the db query. Now we return LoadResult.Error instead which is propagated as a LoadStateUpdate. This also makes it easier to recover from errors.

Test: ./gradlew room:room-paging:cC
Bug: 302708983
Relnote: "Exceptions thrown from PagingSource loads will now be propagated as a LoadStateUpdate of LoadResult.Error containing the Throwable. This error state is observable through PagingDataAdapter.loadStateFlow(Views) or LazyPagingItems.loadState(Compose). Note that this marks a behavioral change where in the past load errors will bubble up as an Exception thrown by the dao method that triggered the load."
(cherry picked from https://android-review.googlesource.com/q/commit:e12e6ff2dd9421be11d2af72e603ad279bb0dfb7)
Merged-In: I93887de62a4fc76d1abe4ade60ed524a41d9d4e8
Change-Id: I93887de62a4fc76d1abe4ade60ed524a41d9d4e8