refactor: inline single-use getcached wrapper
This commit is contained in:
@@ -131,10 +131,6 @@ func (s *EpisodeService) markFailure(ctx context.Context, anime domain.Anime, ca
|
|||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
func (s *EpisodeService) getCached(ctx context.Context, anime domain.Anime) (domain.CanonicalEpisodeList, bool) {
|
|
||||||
return s.getDecodedCached(ctx, anime)
|
|
||||||
}
|
|
||||||
|
|
||||||
func (s *EpisodeService) getFreshCached(ctx context.Context, anime domain.Anime) (domain.CanonicalEpisodeList, bool) {
|
func (s *EpisodeService) getFreshCached(ctx context.Context, anime domain.Anime) (domain.CanonicalEpisodeList, bool) {
|
||||||
row, err := s.queries.GetEpisodeAvailabilityCache(ctx, int64(anime.MalID))
|
row, err := s.queries.GetEpisodeAvailabilityCache(ctx, int64(anime.MalID))
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
|||||||
@@ -141,7 +141,7 @@ func (s *EpisodeService) refresh(ctx context.Context, anime domain.Anime) (domai
|
|||||||
providerAvailability, source, providerErr := s.fetchProviderAvailability(ctx, anime)
|
providerAvailability, source, providerErr := s.fetchProviderAvailability(ctx, anime)
|
||||||
if providerErr != nil {
|
if providerErr != nil {
|
||||||
s.markFailure(ctx, anime, providerErr)
|
s.markFailure(ctx, anime, providerErr)
|
||||||
if cached, ok := s.getCached(ctx, anime); ok {
|
if cached, ok := s.getDecodedCached(ctx, anime); ok {
|
||||||
observability.Warn(
|
observability.Warn(
|
||||||
"episodes_provider_failed_serving_stale_cache",
|
"episodes_provider_failed_serving_stale_cache",
|
||||||
"episodes",
|
"episodes",
|
||||||
|
|||||||
Reference in New Issue
Block a user