[RFC 1/4] Automatically remove old events
Daniel Axtens
dja at axtens.net
Mon Mar 12 19:49:51 AEDT 2018
Daniel Axtens <dja at axtens.net> writes:
> Stephen Finucane <stephen at that.guru> writes:
>
>> On Thu, 2018-01-25 at 16:21 +1100, Daniel Axtens wrote:
>>> Hi Stephen,
>>>
>>> This is a good idea and I'm happy for it to go in as-is, before the
>>> rest
>>> of the series.
>>>
>>> Reviewed-by: Daniel Axtens <dja at axtens.net>
>>
>> I've been thinking a little more on this and I don't know if I actually
>> want to do this. If we manage to solve the performance impact, we can
>> use events as an auditor and display things like state changes on the
>> patch detail page. I might leave this one for another while. Would
>> still appreciate reviews on the rest of this series though.
>
> I think having this configurable would be the best. Perhaps have
> 0 = disable, 1+ = delete after N days.
>
> (Maybe we need to expose a unique ID for patches to help people who use
> them.)
As it turns out, we already do this.
>
> Regards,
> Daniel
>
>>
>> Stephen
>>
>>>
>>> Regards,
>>> Daniel
More information about the Patchwork
mailing list