Github user dsmiley commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/501#discussion_r234646277
  
    --- Diff: 
solr/core/src/java/org/apache/solr/update/DirectUpdateHandler2.java ---
    @@ -975,6 +976,14 @@ private void updateDocOrDocValues(AddUpdateCommand 
cmd, IndexWriter writer) thro
         }
       }
     
    +  boolean forciblyAddBlockId() {
    --- End diff --
    
    Come to think of it, who cares if root is populated or not for "old" 
schemas?  Lets just do it always; no conditional test.  If someone doesn't want 
root to be populated, that same person probably doesn't have child docs and 
should just as well omit the root field from their schema.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to