GROUP BY only model id.

[balhoff]

Use aggregates for everything else, in order to align number of results returned with the number of matching models. Addresses #370

[tmushayahama]

@balhoff I tested and it seems to be working and @lpalbou #370 now brings correct count. However, the reference (PMID search) is now slow.

@kltm can you merge this so that @ukemi @vanaukenk can doo more tests

I will continue to test as see if no side effect

[lpalbou]

It the counts are now correct, that's good enough for me ! Thanks everyone

[lpalbou]

@balhoff do you think this fix will slow down the search ?

@tmushayahama while this gets deployed to prod, if you want to do more tests, the test set should contain large models with several occurrences of the same terms - that's where the search was failing due to limit/pagination on occurrences rather than models. Thanks

[kltm]

Ready to go to dev.

[balhoff]

@balhoff do you think this fix will slow down the search ?

I don't think so.

[tmushayahama]

@balhoff @lpalbou I think there is now a bug that the results are not sorted by date DESC. I think now its hard to find models

Then search by contributor sometimes its giving back 0 results but there is a count. And also multiple contributor
http://barista-dev.berkeleybop.org/search/models?offset=250&limit=50&contributor=http://orcid.org/0000-0002-2874-6934&expand gives n = 0 models
http://barista-dev.berkeleybop.org/search/models?offset=250&limit=50&contributor=http://orcid.org/0000-0002-2874-6934&count = n = 7