[PATCH] Added code to output the age as seconds instead of "0 min."

Ferry Huberts mailings at hupie.com
Sun Sep 30 07:51:02 CEST 2012



On 29-09-12 20:43, Luke SanAntonio wrote:
> Hi Jason,
>
>      First of all, good call with the cache, it hadn't even crossed my mind...
> Second I think the cache isn't something we need to worry about...
> mostly because
> cgit is very lightweight, and I think for now, we don't have to be too
> worried about
> performance, correct me if I'm wrong though... Also it seems to me


I'd like to correct you on that.
Try running a server with many big git repositories with a cgit 
interface on top, that is heavily used by people.
You'll really want to have caching.
Caching is not something I would like to see dropped or non-functional

> that with or without the cache,
> every cgit page I've ever come across seems to load in much less time
> than a second...
>
>
> Thanks for considering my patch =D
>   - Luke
>
> On Sat, Sep 29, 2012 at 1:20 AM, Jason A. Donenfeld <Jason at zx2c4.com> wrote:
>> Hey Luke,
>>
>> Looks great, merged.
>>
>> My one concern, however, which might make me reconsider merging this
>> patch is -- what about the cache? Adding explicit seconds makes any
>> caching appear even more out of date than just minutes. But maybe this
>> isn't a big deal. What do you think?
>>
>> Jason
>
> _______________________________________________
> cgit mailing list
> cgit at hjemli.net
> http://hjemli.net/mailman/listinfo/cgit
>

-- 
Ferry Huberts




More information about the CGit mailing list