Skip to content

Revert "[ESQL] Introduce pluggable external datasource framework (#141678)#142663

Merged
smalyshev merged 2 commits intoelastic:mainfrom
smalyshev:revert_ce37e1ae60974af1a6664a56f0479b1c3d354111
Feb 19, 2026
Merged

Revert "[ESQL] Introduce pluggable external datasource framework (#141678)#142663
smalyshev merged 2 commits intoelastic:mainfrom
smalyshev:revert_ce37e1ae60974af1a6664a56f0479b1c3d354111

Conversation

@smalyshev
Copy link
Contributor

@smalyshev smalyshev commented Feb 18, 2026

This reverts commit 6acdb30. It's causing HeapAttackIT to fail in serverless. On some machines. But not others! How exciting.

For now, we'll revert and then try and get it back more slowly.

For context: this fails for me reliably on serverless main:

./gradlew ":qa:stateful:test:external-modules:test-esql-heap-attack:javaRestTest" --tests "org.elasticsearch.xpack.esql.heap_attack.HeapAttackIT"  -Dtests.iters=2 -Dtests.seed=546B5A9612B7E079

Fails with different seeds too. Does not fail with this revert.

…stic#141678)"

This reverts commit 6acdb30. It's
causing HeapAttackIT to fail in serverless. On some machines. But not
others! How exciting.

For now, we'll revert and then try and get it back more slowly.
@smalyshev smalyshev added >non-issue test-release Trigger CI checks against release build labels Feb 18, 2026
@smalyshev smalyshev force-pushed the revert_ce37e1ae60974af1a6664a56f0479b1c3d354111 branch from 9e80fce to b0b4b0f Compare February 18, 2026 21:57
@smalyshev smalyshev added :Analytics/ES|QL AKA ESQL and removed test-release Trigger CI checks against release build labels Feb 18, 2026
@smalyshev smalyshev changed the title Revert ce37e1ae60974af1a6664a56f0479b1c3d354111 Revert "[ESQL] Introduce pluggable external datasource framework (#141678) Feb 18, 2026
@smalyshev smalyshev marked this pull request as ready for review February 18, 2026 23:05
@smalyshev smalyshev requested a review from a team as a code owner February 18, 2026 23:05
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Feb 18, 2026
@elasticsearchmachine
Copy link
Collaborator

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

@smalyshev
Copy link
Contributor Author

@elasticsearchmachine test this please

@elasticsearchmachine elasticsearchmachine added the serverless-linked Added by automation, don't add manually label Feb 18, 2026
@smalyshev smalyshev merged commit 8f68c0b into elastic:main Feb 19, 2026
37 of 42 checks passed
@smalyshev smalyshev deleted the revert_ce37e1ae60974af1a6664a56f0479b1c3d354111 branch February 19, 2026 00:49
@@ -59,12 +59,13 @@ setup:
- cosine_vector_similarity_function
- inline_stats
- promql_command_v0
- uri_parts_command
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@smalyshev why is this added here and the counter decreased?
I am adding a now command (and capability) now and I am uncertain as to what is expected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL >non-issue serverless-linked Added by automation, don't add manually Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments