Skip to content

[ESQL] Fix async logging consistency and severity#142401

Merged
MattAlp merged 11 commits intoelastic:mainfrom
MattAlp:fix-esql-async-logging
Feb 24, 2026
Merged

[ESQL] Fix async logging consistency and severity#142401
MattAlp merged 11 commits intoelastic:mainfrom
MattAlp:fix-esql-async-logging

Conversation

@MattAlp
Copy link
Copy Markdown
Contributor

@MattAlp MattAlp commented Feb 12, 2026

Summary

  • Ensure async ESQL requests log post-timeout failures and partial failures through the same status-aware paths used by synchronous request handling.
  • Classify async result-store failures by status so 5xx/internal failures are logged at ERROR while expected/non-server failures remain WARN (with DocumentMissing and VersionConflict still suppressed).
  • Add focused tests for timeout callback invocation and failure status classification to guard this behavior.

Related

@MattAlp MattAlp added >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL labels Feb 12, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Hi @MattAlp, I've created a changelog YAML for you.

@MattAlp
Copy link
Copy Markdown
Contributor Author

MattAlp commented Feb 12, 2026

current branch is showing a hanging CI - click through to Buildkite and you'll see that everything passes ☹️

@MattAlp MattAlp enabled auto-merge (squash) February 24, 2026 17:13
@MattAlp MattAlp merged commit d62e850 into elastic:main Feb 24, 2026
35 checks passed
szybia added a commit to szybia/elasticsearch that referenced this pull request Feb 24, 2026
…on-sliced-reindex

* upstream/main:
  Mute org.elasticsearch.reindex.management.ReindexManagementClientYamlTestSuiteIT test {yaml=reindex/30_cancel_reindex/Cancel running reindex returns response and GET confirms completed} elastic#142079
  [ESQL] Fix async logging consistency and severity (elastic#142401)
  more consistent allow_no_indices=false errors (elastic#142373)
  Attribute ES|QL shard search load in Lucene operators (elastic#142841)
  MetricsInfoOperator refactoring (elastic#142935)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL >bug 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.

ESQL: Correctly log errors when storing async results ESQL: inconsistent logging of partial failures in async vs. sync requests

3 participants