Closed
Bug 1036175
Opened 10 years ago
Closed 10 years ago
Update monolith-aggregator to use official elasticsearch libs
Categories
(Marketplace Graveyard :: Statistics, defect, P4)
Tracking
(Not tracked)
RESOLVED
FIXED
2014-07-29
People
(Reporter: robhudson, Assigned: robhudson)
Details
(Whiteboard: [repoman][qa-])
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+.
Updated•10 years ago
|
Priority: -- → P3
Assignee | ||
Comment 1•10 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 | ||
Comment 2•10 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
Closed: 10 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.
Description
•