Skip to content

Commit 1920f00

Browse files
committed
Make NestedObjectMapper its own class (elastic#74410)
Nested objects are implemented via a Nested class directly on object mappers, even though nested and non-nested objects have quite different semantics. In addition, most call-sites that need to get an object mapper in fact need a nested object mapper. To make it clearer that nested and object mappers are different beasts with different implementations and different requirements, we should split them into different classes. server:test compiles
1 parent cf575f4 commit 1920f00

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

server/src/main/java/org/elasticsearch/index/mapper/RootObjectMapper.java

-1
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,6 @@ private static void fixRedundantIncludes(ObjectMapper objectMapper, boolean pare
130130
child.setIncludeInParent(true);
131131
child.setIncludeInRoot(false);
132132
}
133-
fixRedundantIncludes(child, includeInRootViaParent || includedInRoot);
134133
}
135134
}
136135
}

0 commit comments

Comments
 (0)