Skip to content

Commit

Permalink
fix: Compute aggregate on startup if needed
Browse files Browse the repository at this point in the history
Signed-off-by: Vincent Boutour <[email protected]>
  • Loading branch information
ViBiOh committed Dec 30, 2023
1 parent adf06a8 commit f4187a9
Showing 1 changed file with 2 additions and 12 deletions.
14 changes: 2 additions & 12 deletions pkg/metadata/aggregate.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,7 @@ func (s Service) GetMetadataFor(ctx context.Context, item absto.Item) (metadata
ctx, end := telemetry.StartSpan(ctx, s.tracer, "get_metadata")
defer end(&err)

metadata, err = s.exifCache.Get(ctx, item)
if absto.IsNotExist(err) {
err = nil
}

return metadata, err
return s.exifCache.Get(ctx, item)
}

func (s Service) GetAllMetadataFor(ctx context.Context, items ...absto.Item) (map[string]provider.Metadata, error) {
Expand Down Expand Up @@ -68,12 +63,7 @@ func (s Service) GetAggregateFor(ctx context.Context, item absto.Item) (aggregat
ctx, end := telemetry.StartSpan(ctx, s.tracer, "get_aggregate")
defer end(&err)

aggregate, err = s.aggregateCache.Get(ctx, item)
if absto.IsNotExist(err) {
err = nil
}

return aggregate, err
return s.aggregateCache.Get(ctx, item)
}

func (s Service) GetAllAggregateFor(ctx context.Context, items ...absto.Item) (map[string]provider.Aggregate, error) {
Expand Down

0 comments on commit f4187a9

Please sign in to comment.