Skip to content

Rename patterned_text to pattern_text#134834

Merged
parkertimmins merged 12 commits intoelastic:mainfrom
parkertimmins:parker/pattern-text-rename
Sep 17, 2025
Merged

Rename patterned_text to pattern_text#134834
parkertimmins merged 12 commits intoelastic:mainfrom
parkertimmins:parker/pattern-text-rename

Conversation

@parkertimmins
Copy link
Copy Markdown
Contributor

Rename all usage of patterned_text to pattern_text. This includes all classes/files that are named PatternedText*. It also updates the feature flag to pattern_text.

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Copy Markdown
Contributor

@jordan-powers jordan-powers left a comment

Choose a reason for hiding this comment

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

Couple of notes, overall LGTM 👍

@parkertimmins parkertimmins merged commit 56fea57 into elastic:main Sep 17, 2025
34 checks passed
gmjehovich pushed a commit to gmjehovich/elasticsearch that referenced this pull request Sep 18, 2025
Rename all usage of patterned_text to pattern_text. This includes all classes/files that are named PatternedText*. It also updates the feature flag to pattern_text. Only the NodeFeature mapper.patterned_text and the setting index.mapping.patterned_text.disable_templating is not changed.
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.

4 participants