Update monolith-aggregator to use official elasticsearch libs

RESOLVED FIXED in 2014-07-29

Status

Marketplace
Statistics
P4
normal
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: robhudson, Assigned: robhudson)

Tracking

Avenir
2014-07-29
x86
Mac OS X
Points:
---

Details

(Whiteboard: [repoman][qa-])

(Assignee)

Description

4 years ago
Monolith is using Python requests to talk to Elasticsearch if I remember correctly. We could either update it to use the official elasticsearch lib or keep using requests. Either way we can review the queries we're making and see if any can be more optimal on 1.0+.
Priority: -- → P3
(Assignee)

Comment 1

4 years ago
The monolith-aggregator uses pyelasticsearch.  It’s generating its queries using raw python dictionaries.

The monolith-client uses the requests library to send queries to ES.

Updating this bug to switch monolith-aggregator to the official elasticsearch lib, although priority is lower since it shouldn't be affected by the Elasticsearch 1.0+ upgrade.
Priority: P3 → P4
Summary: Check monolith for Elasticsearch 1.0+ support → Update monolith-aggregator to use official elasticsearch libs
Whiteboard: [repoman]
(Assignee)

Updated

4 years ago
No longer blocks: 971801
(Assignee)

Comment 2

4 years ago
I went ahead and did this while working on other monolith-aggregator stuff:
https://github.com/mozilla/monolith-aggregator/commit/ffbfd89c
Assignee: nobody → robhudson.mozbugs
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Whiteboard: [repoman] → [repoman][qa-]
Target Milestone: --- → 2014-07-29
You need to log in before you can comment on or make changes to this bug.