Skip to content

Conversation

@jordan-powers
Copy link
Contributor

Looks like #131317 had been been ready but not merged for several weeks, during which updates were made in main that broke the build, and the CI was not re-run before merging. This PR fixes the build issue.

@jordan-powers jordan-powers requested a review from Kubik42 August 21, 2025 18:57
@elasticsearchmachine elasticsearchmachine added v9.2.0 needs:triage Requires assignment of a team area label labels Aug 21, 2025
@jordan-powers jordan-powers added >non-issue :StorageEngine/Mapping The storage related side of mappings and removed needs:triage Requires assignment of a team area label labels Aug 21, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

Copy link
Contributor

@Kubik42 Kubik42 left a comment

Choose a reason for hiding this comment

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

looks good!

@Kubik42
Copy link
Contributor

Kubik42 commented Aug 21, 2025

Going to merge, everything looks good. The pr-upgrade failures are due to the fact that we're attempting to build main, which is currently failing. This PR is meant to address that.

@Kubik42 Kubik42 merged commit f5759b1 into elastic:main Aug 21, 2025
26 of 33 checks passed
@jordan-powers jordan-powers deleted the fix-compile-text-field-builder branch September 2, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants