Skip to content

ESQL: Fix InvalidMappedField equality metadata - #149151

Closed
mturac wants to merge 2 commits into
elastic:mainfrom
mturac:fix/issue-145907
Closed

mturac wants to merge 2 commits into
elastic:mainfrom
mturac:fix/issue-145907

Conversation

@mturac

@mturac mturac commented May 15, 2026

Copy link
Copy Markdown

Closes #145907

This updates InvalidMappedField so equality and hash code include the conflict metadata that distinguishes different mapped field conflicts. Error messages for type-conflict metadata are derived when needed instead of being eagerly stored.

Tests:

  • ./gradlew :x-pack:plugin:esql:test --tests org.elasticsearch.xpack.esql.type.InvalidMappedFieldTests
  • ./gradlew :x-pack:plugin:esql:spotlessJavaCheck

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.5.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels May 15, 2026
@kingherc kingherc added the :Analytics/ES|QL AKA ESQL label May 20, 2026
@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) and removed needs:triage Requires assignment of a team area label labels May 20, 2026
@elasticsearchmachine

Copy link
Copy Markdown
Collaborator

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

@alex-spies

Copy link
Copy Markdown
Contributor

Thank you for your contribution @mturac !

Unfortunately, the issue has already been addressed in #146117.

@alex-spies alex-spies closed this May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ESQL: InvalidMappedField has bad equals, hashCode and should use a lazy error message

4 participants