913811 - HTTP cache v2: store frecency to cache entries

Status: RESOLVED DUPLICATE of bug 924116

(Core :: Networking: Cache, defect)

Description

[Honza Bambas (:mayhemer)]

I think of adding a new specialized properties to CacheFileMetadata beside Set/GetExpirationTime.

Comment 1

[Honza Bambas (:mayhemer)]

Attachment: v1

Comment 2

[Honza Bambas (:mayhemer)]

Attachment: v2 (persist as int)

Comment 3

[Michal Novotny [:michal]]

Comment on attachment 8336846 [details] [diff] [review]
v2 (persist as int)

Review of attachment 8336846 [details] [diff] [review]:
-----------------------------------------------------------------

::: netwerk/cache2/CacheEntry.cpp
@@ +337,5 @@
>  
> +    uint32_t frecency;
> +    mFile->GetFrecency(&frecency);
> +    // mFrecency is held in a double to increase computance precision.
> +    // It is ok to persist frecency only as a float (single precision).

float -> uint32_t

Comment 4

[Honza Bambas (:mayhemer)]

Attachment: v2.1

Thanks.

I'll land this after I have a plan for the frecency decay time decision testing patch (bug 924116).

Comment 5

[Honza Bambas (:mayhemer)]

This should land together with bug 924116.