Skip to content

Add false_allow_templates as a dynamic mapping option#944

Open
bruce-hong-glean wants to merge 4 commits intoopensearch-project:mainfrom
bruce-hong-glean:false-allow-templates
Open

Add false_allow_templates as a dynamic mapping option#944
bruce-hong-glean wants to merge 4 commits intoopensearch-project:mainfrom
bruce-hong-glean:false-allow-templates

Conversation

@bruce-hong-glean
Copy link
Copy Markdown

@bruce-hong-glean bruce-hong-glean commented Jul 23, 2025

Description

Add false_allow_templates as a dynamic mapping option. Will hold off on merging until the PR to the main project has merged.

Issues Resolved

opensearch-project/OpenSearch#18617 (comment)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@bruce-hong-glean bruce-hong-glean changed the title Add false_allow_templates as a dynamic mapping opton Add false_allow_templates as a dynamic mapping option Jul 23, 2025
@bruce-hong-glean bruce-hong-glean marked this pull request as ready for review July 23, 2025 20:08
Add test

Signed-off-by: Bruce Hong <bruce.hong@glean.com>
@github-actions
Copy link
Copy Markdown
Contributor

Changes Analysis

Commit SHA: a01d55a
Comparing To SHA: 6501a8a

API Changes

Summary

SUCCESS: No changes found between specifications

Report

The full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/16503264533/artifacts/3621008774

API Coverage

Before After Δ
Covered (%) 666 (65.23 %) 666 (65.23 %) 0 (0 %)
Uncovered (%) 355 (34.77 %) 355 (34.77 %) 0 (0 %)
Unknown 128 128 0

@bruce-hong-glean
Copy link
Copy Markdown
Author

Hi, could I get a review on this now that the main PR has been merged in?

Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Signed-off-by: Bruce Hong <bruce.hong@glean.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant