lusini / elasticsearch-grouping-patches Goto Github PK

hi,

Thanks for your work on this - much better to be able to apply a patch to the current master version, rather than work with a fork.

However, there is an issue with the Search API 'fields' option after applying this patch - if a specific field is specified, the entire _source in returned (ie the fields option is ignored). If the fields array is provided in the query, but left empty, then the entire _source if correctly omitted - but there's no way to only have specified fields returned. This happens with and without a groupField specified in the query, so something seems to have broken ES's normal behaviour when it comes to returning specific fields from the _source document.

elasticsearch-with-local-grouping, currently built with elasticsearch 0.20.beta, has the correct behaviour.

The 'fields' behaviour is included in the patch, so I'm assuming it was supported in version 0.19.4 (I've also successfully applied the path to 0.19.8 - but it has the same issues). I've not tried this patch against 0.20.beta yet.

I'll continue to look into this - though my java knowledge is pretty minimal!

http://www.elasticsearch.org/guide/reference/api/search/fields.html

Jamie.