Page MenuHomePhabricator

Fixup deprecation warnings coming from browser tests in elasticsearch 5
Closed, ResolvedPublic

Description

Not sure which tests exactly these come from, but here is a list. These are not critical, they are only deprecated not removed. We should still get everything up to date though.

[org.elasticsearch.deprecation.common.ParseField] Deprecated field [type] used, replaced by [match_phrase and match_phrase_prefix query]
[org.elasticsearch.deprecation.common.ParseField] Deprecated field [min_word_len] used, expected [min_word_length] instead
[org.elasticsearch.deprecation.common.ParseField] Deprecated field [max_word_len] used, expected [max_word_length] instead
[org.elasticsearch.deprecation.index.mapper.BooleanFieldMapper] searching using boolean value [1] is deprecated, please use [true] or [false]
[org.elasticsearch.deprecation.common.ParseField] Deprecated field [optimize_bbox] used, replaced by [no replacement: optimize_bbox is no longer supported due to recent improvements]

Event Timeline

Change 341025 had a related patch set uploaded (by ebernhardson):
[mediawiki/extensions/CirrusSearch] Fix deprecation notices for norms mapping

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

Change 341026 had a related patch set uploaded (by ebernhardson):
[mediawiki/extensions/CirrusSearch] Rename deprecated more like this fields

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

Change 338032 had a related patch set uploaded (by ebernhardson):
[mediawiki/extensions/GeoData] [DNM] Convert GeoData for elasticsearch 5 update

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

Change 341025 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch] Fix deprecation notices for norms mapping

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

Change 341026 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch] Rename deprecated more like this fields

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

The only remaining deprecation warning comes from something Elastica does itself. I've pushed a patch upstream to fix it (https://github.com/ruflin/Elastica/pull/1263). This shouldn't be a big problem, we only call Type::exists() from maintenance operations, it won't be massive logspam from regular queries.

Change 338032 merged by jenkins-bot:
[mediawiki/extensions/GeoData] Convert GeoData for elasticsearch 5 update

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