Skip to content

Commit 2e2b37f

Browse files
committed
Remove workaround from randomized tests
1 parent 9561108 commit 2e2b37f

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

test/framework/src/main/java/org/elasticsearch/logsdb/datageneration/datasource/DefaultMappingParametersHandler.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,7 @@ private Supplier<Map<String, Object>> keywordMapping(
4949
// We only add copy_to to keywords because we get into trouble with numeric fields that are copied to dynamic fields.
5050
// If first copied value is numeric, dynamic field is created with numeric field type and then copy of text values fail.
5151
// Actual value being copied does not influence the core logic of copy_to anyway.
52-
//
53-
// TODO
54-
// We don't use copy_to on fields that are inside an object with dynamic: strict
55-
// because we'll hit https://github.com/elastic/elasticsearch/issues/113049.
56-
if (request.dynamicMapping() != DynamicMapping.FORBIDDEN && ESTestCase.randomDouble() <= 0.05) {
52+
if (ESTestCase.randomDouble() <= 0.05) {
5753
var options = request.eligibleCopyToFields()
5854
.stream()
5955
.filter(f -> f.equals(request.fieldName()) == false)

0 commit comments

Comments
 (0)