search

elastic / elasticsearch

Free and Open Source, Distributed, RESTful Search Engine
https://www.elastic.co/products/elasticsearch
Other
1.12k stars 24.83k forks source link

ESQL: Re-enable dependency checks #105436

Open costin opened 9 months ago

costin commented 9 months ago

Description

Re-enable the optimizers dependency checkers - see: https://github.com/elastic/elasticsearch/pull/105371#issuecomment-1940377580

elasticsearchmachine commented 9 months ago

Pinging @elastic/es-analytical-engine (Team:Analytics)

alex-spies commented 8 months ago

When re-enabling, we should make sure that a failed dependency check results in a 500 response code sent back to the user; therefore, we can't throw a VerificationException or any other client exception - we could throw an IllegalStateException instead. See discussion here.