pablobm left a comment (openstreetmap/openstreetmap-website#6566)
Unless anyone disagrees (and please do voice it), I intend to leave this as is
and move on to a different piece of work.
On one hand, my implementation is not great as discussed above. There's no
point in using the DB when we don't need a web interface. A static file would
have sufficed and would probably be more performant.
On the other hand, the feature is already in use. Admins have added lists of
phrases to the DB, and these are being used for spam scoring as I write these
lines. Let's wait and see how this works out. Perhaps it's completely useless
and we decide we may as well scrap the whole feature. Or perhaps we think it's
successful, and then we can look into potential improvements, which may include
moving to in-memory data.
Additionally, I experimented a bit with our Chef config, to see what it would
take to get this working with a "data bag", etc. I cannot get the thing to run
(issue filed at https://github.com/openstreetmap/chef/issues/816), so that
would delay me even longer. I don't think it's worth it at this time.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/6566#issuecomment-3626336063
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/issues/6566/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev