tree 777a5bce92abb606725aba07f74a7a64ad36a881
parent 610965bda3e71430e90595c62100004598b654e1
author Clara Fok <clarafok@google.com> 1697502942 -0700
committer Clara Fok <clarafok@google.com> 1697561158 -0700

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."

Change-Id: I93887de62a4fc76d1abe4ade60ed524a41d9d4e8