[openstreetmap/openstreetmap-website] Notes ordering: show unresolved notes first (PR #3386)
Tom Hughes
notifications at github.com
Tue Feb 1 18:59:33 UTC 2022
Well having just been looking at this query for #3443 the fact that we're ordering on `notes` but selecting on `note_comments` is not ideal from a performance point of view given that we have a limit as we have to find all the records and order them before applying the limit.
This change doesn't actually change that though as the problem already exists so it's not a reason not to do this.
More concerning probably is that it won't be obvious to users what the ordering is given the statuses are all grouped together.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/3386#issuecomment-1027182059
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/3386/c1027182059 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20220201/e9a14c85/attachment.htm>
More information about the rails-dev
mailing list