Page MenuHomePhabricator

Update ElasticsearchIntermediary::classifyErrorMessage() for es2.x
Closed, ResolvedPublic

Description

I just noticed while digging through some other things that this function was missed while we were upgrading things. This looks to primarily be used to distinguish between queries that fail and queries that are rejecting, which is then logged to graphite and shows up in our dashboards.

Details

Related Gerrit Patches:
mediawiki/extensions/CirrusSearch : es2.xUpdate classifyError to comply with new error handling

Event Timeline

Restricted Application added projects: Discovery, Discovery-Search. · View Herald TranscriptMay 12 2016, 9:46 PM
Restricted Application added subscribers: Zppix, Aklapper. · View Herald Transcript

Change 288987 had a related patch set uploaded (by DCausse):
Update classifyError to comply with new error handling

https://gerrit.wikimedia.org/r/288987

Change 288987 merged by jenkins-bot:
Update classifyError to comply with new error handling

https://gerrit.wikimedia.org/r/288987

debt closed this task as Resolved.Jun 8 2016, 12:32 AM
debt added a subscriber: debt.

Looks like this is resolved - closing.