talmantel / CheapTripAndroid

0 stars 3 forks source link

Redefine repository strategies #10

Open denis-luttcev opened 3 years ago

denis-luttcev commented 3 years ago

Reason: Identified incorrect work of the caching strategy in the repository and it temporarily change to direct read, issue #8 To speed up the application (especially with a network data source) backup or caching repository strategy is urgent.

Implementation: After add crashlytics (issue #9) return caching strategy on a test environment and analyze crashes. Measure the performance of an application with a direct read strategy. Based on the results, decide on the need for additional repository strategies.

denis-luttcev commented 3 years ago

A possible bug is hidden in the formation of a composite primary key for the 'location' table in the 'LocalDb' database. Reported crashes caused by select two location values when only one was expected.