[ 
https://issues.apache.org/jira/browse/SOLR-5175?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13755733#comment-13755733
 ] 

Yonik Seeley commented on SOLR-5175:
------------------------------------

Although this doesn't seem to be a bug, I was still considering committing it 
because it doesn't reorder children of a particular level.

{code}
Logical view (a tree structure of parents having children):
a
 b
  c
  d
   e
 f


Current index order (everything reversed):
 f
   e
  d
  c
 b
a


Index order of this patch (parents only put at end of children):
  c
   e
  d
 b
 f
a
{code}

Viewed one way, it makes sense to not reorder children (esp when looking at a 
single-level example).  But given that we already must reorder parents with 
respect to their children, not reordering the children actually might confuse 
the multi-level case slightly. 
                
> Don't reorder children document
> -------------------------------
>
>                 Key: SOLR-5175
>                 URL: https://issues.apache.org/jira/browse/SOLR-5175
>             Project: Solr
>          Issue Type: Bug
>          Components: update
>            Reporter: Mikhail Khludnev
>              Labels: patch, test
>             Fix For: 4.5, 5.0
>
>         Attachments: SOLR-5175.patch
>
>
> AddUpdateCommand reverses children documents that causes failure of 
> BJQParserTest.testGrandChildren() discussed in SOLR-5168  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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

Reply via email to