Will Kahn-Greene [:willkg] ET needinfo? me Assignee
	Description • 10 years ago

We should update to ElasticUtils 0.10 for two reasons:

1. At some point, IT will hopefully update our Elasticsearch cluster to 1.0+. ElasticUtils 0.10 supports Elasticsearch 0.90 (what we're using now), 1.0 and 1.1. We need to update before they update our cluster.

2. Oh, wait--only one reason, but it's a big one.


This bug covers updating.

	Will Kahn-Greene [:willkg] ET needinfo? me Assignee
	Comment 1 • 10 years ago

The changes were pretty straight-forward:

1. update ElasticUtils to v0.10

2. change instances where we're using .values_list() or .values_dict() because the shape of the results is slightly different in ElasticUtils v0.10 (all values are now lists)

3. add some code to the API to "reshape" results because we're essentially taking what's in ES and sending it back to the user, but we only want lists as values for fields that should be lists (e.g. "description_bigrams")


In a PR: https://github.com/mozilla/fjord/pull/330

	Will Kahn-Greene [:willkg] ET needinfo? me Assignee
	Comment 2 • 10 years ago

Landed in master in: https://github.com/mozilla/fjord/commit/c4e8e345

	Will Kahn-Greene [:willkg] ET needinfo? me Assignee
	Comment 3 • 10 years ago

Fixed a bug I found on stage: https://github.com/mozilla/fjord/commit/e023fa4e

Pushed to production just now.