Add prefer_v2_templates parameter to Reindex#56253
Add prefer_v2_templates parameter to Reindex#56253probakowski merged 6 commits intoelastic:masterfrom
Conversation
|
Pinging @elastic/es-core-features (:Core/Features/Indices APIs) |
|
@elasticmachine update branch |
|
user doesn't have permission to update head repository |
|
@elasticmachine update branch |
|
user doesn't have permission to update head repository |
dakrone
left a comment
There was a problem hiding this comment.
Thanks for doing this @probakowski! I left two really minor comments that are just related to BWC for the request, otherwise this looks good to me!
server/src/main/java/org/elasticsearch/index/reindex/ReindexRequest.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/elasticsearch/index/reindex/ReindexRequest.java
Outdated
Show resolved
Hide resolved
| } | ||
|
|
||
| if (request.hasParam("prefer_v2_templates")) { | ||
| internal.preferV2Templates(Boolean.parseBoolean(request.param("prefer_v2_templates"))); |
There was a problem hiding this comment.
for 8.0 we want this to default to true if it's not specified by the user (this is just a reminder to change this after backport)
There was a problem hiding this comment.
I wonder if we should change it here or simply pass null down the road and let create index request pick up correct default (for making it DRY and using only single place to set defaults)
|
@elasticmachine run elasticsearch-ci/2 |
* prefer_v2_templates for reindex
Relates to #53101