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

Hoss Man commented on SOLR-3535:
--------------------------------

bq. Or simply allow SolrInputDocument as a normal value and existing APIs could 
be used to add them.  This would also be slightly more powerful, allowing more 
than one child list for the same parent.

"allow SolrInputDocument as a normal value" ... a normal value to what? where? 
... are you describing he same thing as Mikhail in modeling "children" 
SolrInputDocuments as field values of the parent SOlrInputDOcument?

If so then i ask you the same questions i asked him above...

{quote}
bq. why new api/property is necessary? is solrInputDoc.addField("skus", new 
Object[]\{sku1, sku2, sku3\}) not enough?

Are you suggesting we model child documents as objects (SolrInputDocuments i 
guess?) in a special field? ... what if i put child documents in multiple 
fields? would that signify the different types of child? how would solr model 
that in the (lucene) Documents when giving them to the InddexWriter? How would 
solr know how to order the children in from multiple fields/lists when creating 
the block? Wouldn't the "type of child" information be better living in the 
child documents itself? (particularly since that "type" information needs to be 
in the child documents anyway so that the filter query for a BJQ can be 
specified.)

It also seems like it would require code that wants to know what children exist 
in a document to do a lot of work to find that out (need to iterate ever field 
in the SolrInputDocument and do reflection to see if they are child-documents 
or not)

Another concern off the top of my head is that a lot of existing code 
(including any custom update processors people might have) would assume those 
child documents are multivaluved field values and would probably break – hence 
a new method on SolrInputDocument seems wiser (code that doens't know about may 
not do what you want, but at least it won't break it)
{quote}
                
> Add block support for XMLLoader
> -------------------------------
>
>                 Key: SOLR-3535
>                 URL: https://issues.apache.org/jira/browse/SOLR-3535
>             Project: Solr
>          Issue Type: Sub-task
>          Components: update
>    Affects Versions: 4.1, 5.0
>            Reporter: Mikhail Khludnev
>            Priority: Minor
>         Attachments: SOLR-3535.patch
>
>
> I'd like to add the following update xml message:
> <add-block>
>     <doc>....</doc>
>     <doc>....</doc>
> </add-block>
> out of scope for now: 
> * other update formats
> * update log support (NRT), should not be a big deal
> * overwrite feature support for block updates - it's more complicated, I'll 
> tell you why
> Alt
> * wdyt about adding attribute to the current tag {pre}<add block="true">{pre} 
> * or we can establish RunBlockUpdateProcessor which treat every <add> 
> ....</add> as a block.
> *Test is included!!*
> How you'd suggest to improve the patch?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to