Fixes fetching only specific fields, including only _id.

ChristopherRabotin / bungiesearch

UNMAINTAINED CODE -- Elasticsearch-dsl-py django wrapper with mapping generator

BSD 3-Clause "New" or "Revised" License

67 stars 20 forks source link

Fixes fetching only specific fields, including only _id. #84

Closed ChristopherRabotin closed 9 years ago

ChristopherRabotin commented 9 years ago

Fixed #81.

ChristopherRabotin commented 9 years ago

@afrancis13, do you think these newly added tests cover the case of specific primary keys well enough? I'm asking specifically because this PR fixes raw result mapping when one wants elasticsearch to return as little information as possible (hence fields('_id')) and rely solely on the database to fetch the right content.

afrancis13 commented 9 years ago

I pulled it down and played with it for a little while. This looks good to me at first glance, I'll let you know if i have issues in the future, and will submit a small PR in that case.