Github user moshebla commented on a diff in the pull request:
https://github.com/apache/lucene-solr/pull/416#discussion_r213306038
--- Diff:
solr/core/src/java/org/apache/solr/response/transform/ChildDocTransformer.java
---
@@ -109,9 +109,14 @@ public void transform(SolrDocument rootDoc, int
rootDocId) {
// Loop each child ID up to the parent (exclusive).
for (int docId = calcDocIdToIterateFrom(lastChildId, rootDocId);
docId < rootDocId; ++docId) {
- // get the path. (note will default to ANON_CHILD_KEY if not in
schema or oddly blank)
+ // get the path. (note will default to ANON_CHILD_KEY if schema
is not nested or empty string if blank)
String fullDocPath = getPathByDocId(docId - segBaseId,
segPathDocValues);
+ if(isNestedSchema && !fullDocPath.contains(transformedDocPath)) {
+ // is not a descendant of the transformed doc, return fast.
+ return;
--- End diff --
Yep, you're right.
I'll investigate further to see why a test did not fail because of this.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
