search

scripting / a8c-FeedLand-Support

A public repo for discussing FeedLand at A8C.
1 stars 0 forks source link

Indexes #54

Open scripting opened 11 months ago

scripting commented 11 months ago

There's at least one index that should be added, for email addresses in the users table, we look up users as often by email as we do by screenname.

Also Scott Hanson wrote about an index that dramatically decreased the number of rows his instance of FeedLand looked at in queries. (Just looked for it couldn't find quickly.)

In general we should be talking about the database regularly, and encourage people who know SQL to review the code, and suggest tweaks.

fmfernandes commented 11 months ago

This is the index improvement you mentioned: https://github.com/scripting/feedlandInstall/issues/41

I tried it locally and indeed the number of rows searched decreased by a lot. I can't think of a reason not to have that on default FeedLand installs 👍🏻

scripting commented 10 months ago

Added an index so isItemInDatabase isn't so wasteful. Recommended by Scott Hanson and Fernando.

https://github.com/scripting/feedlandInstall/issues/41#issuecomment-1810338879