rietveld-codereview / rietveld

Code Review, hosted on Google App Engine
https://codereview.appspot.com
Apache License 2.0
560 stars 152 forks source link

Not all possible searches have composite indexes in index.yaml. #521

Closed qyearsley-zz closed 9 years ago

qyearsley-zz commented 9 years ago

I was just using https://codereview.chromium.org/search, and found that some searches work but others don't.

For example, when I try filtering by "closed" and "owner" and ordering by "modified" from old to new: https://codereview.chromium.org/search?closed=2&owner=robertocn%40chromium.org&order=modified&limit=60

Details: no matching index found. The suggested index for this query is:

maybe there are some more possible combinations that are missing indexes in index.yaml?

andialbrecht commented 9 years ago

Please note, this is fixed on the default branch (codereview.appspot.com). To have this index on codereview.chromium.org you'll have to wait until the chromium branch is updated from the default and a new version is deployed on codereview.chromium.org.

qyearsley-zz commented 9 years ago

Thanks!