Re: [PR] merge DecodeShortURLs and Redirectors plugins [sp amassassin]

bigio commented on PR #36:
URL: https://github.com/apache/spamassassin/pull/36#issuecomment-5413706111

   > Perhaps we could keep the older schema (short_url_cache) and deprecate the newer schema (redir_url_cache) since there's probably fewer people using the newer schema.
   > 
   if we suggest using Redirectors plugin I think we should keep cache tables with consistent names,
   since it's cache, even starting with an empty cache might be doable.
   
   > We could also include a migration script but for a cache with a 24 hour TTL, it's not really worth migrating anything. You just need to make sure the table exists.
   
   I agree, adding a migration script on `sql/` is a useful task.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]