UEWBot / dipvis

Django-based visualiser for tournaments for the boardgame Diplomacy
GNU General Public License v3.0
7 stars 4 forks source link

Remove unnecessary Meta.orderings #258

Open UEWBot opened 1 year ago

UEWBot commented 1 year ago

They make sense when we always want a specific ordering, but not when we need different orderings in different places. Need to audit the code for all uses of order_by() and all Meta.ordering and check that they all make sense. The tricky part is that we also need to find all the places where we rely on the default (Meta) ordering, which may need to become explicit instead. If we can get rid of SQL ORDER_BY clauses, that would help performance.