Revisions and Commits
|D12967||rP7ecd5155e52f elasticsearch host quick fix|
|D12948||rPf5a9d1f8d47b Raise a setup issue for misconfigured Elasticsearch|
rP16a8ed72bdda: Modernize search engine selection actually broke the elasticsearch integration. The Selector class that was removed did the initialisation of the ElasticSearchEngine (setting the elastic.host and namespace) which is completely missing now so the executeRequest just calls /_status/ without any protocol/domain which results in the malformed url exception.
Attached diff would be a quick fix. Probably not the right place to put it though. There seems to be no good place to actually call the new setURI/setIndex
Presuming this is resolved by D12967.
The way the ElasticSearch engine class works is still a little odd (it is sort of undecided about whether it is a general-purpose, loosely-coupled, configurable class or a tightly-coupled, auto-configured class) but this concern is largely philosophical and I don't think it needs to be addressed until we're touching this part of the code again.