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

David Eric Pugh commented on SOLR-15152:
----------------------------------------

Hi all, I believe the attached PR is now ready for review.   

I'm pretty happy with everything EXCEPT some changes I had to make to 
{{ChildDocTransformerFactory.processPathHierarchyQueryString()}}..   I added 
some checks to decide when to escape the {{childFilter}} query string, but I 
did them kind of blindly, just to get the tests to pass :(.   I would love 
[~dsmiley] or others to look here.

I'm looking forward to being able to roundtrip export and import Solr docs that 
have children successfully!

> Export Tool should export nested docs cleanly in .json, .jsonl, and javabin
> ---------------------------------------------------------------------------
>
>                 Key: SOLR-15152
>                 URL: https://issues.apache.org/jira/browse/SOLR-15152
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: SolrCLI
>    Affects Versions: 8.8
>            Reporter: David Eric Pugh
>            Assignee: David Eric Pugh
>            Priority: Major
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> ExportTool doesn't properly handle anonymous child docs or nested docs.   It 
> also confuses the JSONL format with the JSON format.  
> I'd like to have the JSON Lines format output as .jsonl, which is the 
> standard, and have the JSON format to be a .json, which is the same output as 
> if you wanted to post a Solr doc as a JSON to upload the data...    This will 
> let us round trip the data.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to