[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-09-02 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit 679abb76808cb0e0fd0ea4b2bab0e92d3073d6a4 in lucene-solr's branch 
refs/heads/branch_8x from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=679abb7 ]

SOLR-14383: Fix nested indexing ref-guide documentation and corrisponding 
search examples

Squashed commit of branch jira/SOLR-14383 as of...

commit 34da84c238f3aff57cd7bba8040ccf6f6d94cf15
Merge: 5d293276a34 8bd79ec301f
Author: Chris Hostetter 
Date:   Tue Sep 1 16:39:03 2020 -0700

Merge branch 'master' into jira/SOLR-14383

(cherry picked from commit a70a47d053954b0f13b7da6f5612432a8ea7a898)


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch, SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-09-02 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit 679abb76808cb0e0fd0ea4b2bab0e92d3073d6a4 in lucene-solr's branch 
refs/heads/branch_8x from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=679abb7 ]

SOLR-14383: Fix nested indexing ref-guide documentation and corrisponding 
search examples

Squashed commit of branch jira/SOLR-14383 as of...

commit 34da84c238f3aff57cd7bba8040ccf6f6d94cf15
Merge: 5d293276a34 8bd79ec301f
Author: Chris Hostetter 
Date:   Tue Sep 1 16:39:03 2020 -0700

Merge branch 'master' into jira/SOLR-14383

(cherry picked from commit a70a47d053954b0f13b7da6f5612432a8ea7a898)


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch, SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-09-02 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit 679abb76808cb0e0fd0ea4b2bab0e92d3073d6a4 in lucene-solr's branch 
refs/heads/branch_8x from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=679abb7 ]

SOLR-14383: Fix nested indexing ref-guide documentation and corrisponding 
search examples

Squashed commit of branch jira/SOLR-14383 as of...

commit 34da84c238f3aff57cd7bba8040ccf6f6d94cf15
Merge: 5d293276a34 8bd79ec301f
Author: Chris Hostetter 
Date:   Tue Sep 1 16:39:03 2020 -0700

Merge branch 'master' into jira/SOLR-14383

(cherry picked from commit a70a47d053954b0f13b7da6f5612432a8ea7a898)


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch, SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-09-02 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit a70a47d053954b0f13b7da6f5612432a8ea7a898 in lucene-solr's branch 
refs/heads/master from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=a70a47d ]

SOLR-14383: Fix nested indexing ref-guide documentation and corrisponding 
search examples

Squashed commit of branch jira/SOLR-14383 as of...

commit 34da84c238f3aff57cd7bba8040ccf6f6d94cf15
Merge: 5d293276a34 8bd79ec301f
Author: Chris Hostetter 
Date:   Tue Sep 1 16:39:03 2020 -0700

Merge branch 'master' into jira/SOLR-14383


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch, SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-09-02 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit a70a47d053954b0f13b7da6f5612432a8ea7a898 in lucene-solr's branch 
refs/heads/master from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=a70a47d ]

SOLR-14383: Fix nested indexing ref-guide documentation and corrisponding 
search examples

Squashed commit of branch jira/SOLR-14383 as of...

commit 34da84c238f3aff57cd7bba8040ccf6f6d94cf15
Merge: 5d293276a34 8bd79ec301f
Author: Chris Hostetter 
Date:   Tue Sep 1 16:39:03 2020 -0700

Merge branch 'master' into jira/SOLR-14383


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch, SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-09-02 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit a70a47d053954b0f13b7da6f5612432a8ea7a898 in lucene-solr's branch 
refs/heads/master from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=a70a47d ]

SOLR-14383: Fix nested indexing ref-guide documentation and corrisponding 
search examples

Squashed commit of branch jira/SOLR-14383 as of...

commit 34da84c238f3aff57cd7bba8040ccf6f6d94cf15
Merge: 5d293276a34 8bd79ec301f
Author: Chris Hostetter 
Date:   Tue Sep 1 16:39:03 2020 -0700

Merge branch 'master' into jira/SOLR-14383


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch, SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-09-01 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit 34da84c238f3aff57cd7bba8040ccf6f6d94cf15 in lucene-solr's branch 
refs/heads/jira/SOLR-14383 from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=34da84c ]

Merge branch 'master' into jira/SOLR-14383


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-08-21 Thread Chris M. Hostetter (Jira)


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

Chris M. Hostetter commented on SOLR-14383:
---

FWIW: i keep going back and forth on _where_ it makes the most sense to explain 
non-trivial of/which params.

My current thinking, as reflected in my latest branch commits, is that 
{{other-parsers.adoc}} should have a sub section introducing the _concept_ 
(which i've labeled "Block Mask" to differentiate from "all parents" when it 
has to include docs which are not parents or children) that explains how/why it 
can be tricky in non-trivial situations, and then the (later and more in depth) 
{{searching-nested-documents.adoc}} go into the specifics of writing good 
"Block Mask" queries when using {{_nest_path_}} with multi-level hierarchies.

(still work to be done, but i'm about to go offline for a week and didn't want 
to forget this on my laptop)

> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-08-12 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit 3095dd23958aa3dd9190ca698e1258f994ad374c in lucene-solr's branch 
refs/heads/jira/SOLR-14383 from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=3095dd2 ]

SOLR-14383: minor typo/inconsistency fix in sample doc ids

Also add a unit test of hueristic for 'safe' of/which params based on 
_nest_path_ before trying to figure out how to document it clearly

(This is basically the manual 'long form' way of writing child/parent queries 
using the (hypothetical) syntactic sugar described in SOLR-14687)


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-31 Thread David Smiley (Jira)


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

David Smiley commented on SOLR-14383:
-

I really like your name choice of "manuals" -- definitely better than mine.
* "Even though the child documents in these examples are provided syntactically 
as field values syntactically" -- has "syntactically" twice
* (SolrJ) "nocommit: ... do we even have a test proving this works correctly" 
--  Yes we do: 
{{org.apache.solr.update.processor.NestedAtomicUpdateTest#testBlockAtomicStack}}
  there are others I think.
* (SolrJ) "nocommit: the SolrInputDocument methods for addChildDocument methods 
still don't take "field name". -- IMO, I'd rather we simply do away with these 
"child" methods on SolrInputDocument.  The user can add them with a 
field/label: doc.addField("manual", new SolrInputDocument(...)).  If the user 
has simple requirements that doesn't want to pay for the overhead of a nest 
path field that won't be used (albeit minor?), then adding as a field will 
still work (as the ref guide says), it's just that on retrieval, they will be 
on a pseudo-field "_childDocuments_" because the field/label info wasn't 
captured.

> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-31 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit 7086d6765215c1137ccc623468d4925a595d1fea in lucene-solr's branch 
refs/heads/jira/SOLR-14383 from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=7086d67 ]

SOLR-14383: tweaks bassed on feedback from smiley

 - typo fixes

 - using TIP box to draw attention to prefix routing

 - switch from 'producs+skus+docs' to 'products+skus+manuals' for examples


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-31 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit 8423eb9570712489fded77ee98e2f495026a2aac in lucene-solr's branch 
refs/heads/jira/SOLR-14383 from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=8423eb9 ]

SOLR-14383.patch circa 2020-07-30

https://issues.apache.org/jira/secure/attachment/13008628/SOLR-14383.patch


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-31 Thread ASF subversion and git services (Jira)


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

ASF subversion and git services commented on SOLR-14383:


Commit 8423eb9570712489fded77ee98e2f495026a2aac in lucene-solr's branch 
refs/heads/jira/SOLR-14383 from Chris M. Hostetter
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=8423eb9 ]

SOLR-14383.patch circa 2020-07-30

https://issues.apache.org/jira/secure/attachment/13008628/SOLR-14383.patch


> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-31 Thread Chris M. Hostetter (Jira)


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

Chris M. Hostetter commented on SOLR-14383:
---

{quote}Hoss: please use PRs, which are much easier to peer review. I've been 
dragging my feet, not looking forward to reading asciidoc diffs. ...
{quote}
No thank you. I have no interest in using github specific features – 
particularly for ASF projects.

I've created a {{jira/SOLR-14383}} branch for this issue (about to push) if 
that will help lower the bar for your future review.
{quote} * I really appreciate the conversion of plain JSON input docs to a real 
curl equivalent that I could use right away.
 * The bullet advising the ID to be a composite ID for all docs in the block 
really seems like it should be a TIP.
 * Typo: thir -> their
 * Typo: garuntee -> guarantee (multiple)
 * The example about office supplies uses a child doc label of "docs", seen in 
sending the data to Solr and in retrieval. I find the use of this word 
confusing because it's a common word in Solr, so I suspect others might as 
well. Perhaps "documentation" or "files" might be alternatives? "files" is nice 
and short.{quote}
All great points, these edits are all made on the branch – i went with 
"manuals" over "files" because in general (not just in this situation) i feel 
like "files" is _WAY_ too ambiguious in many examples when dealing with Solr 
(ie: "JSON Files" that you index containing metadata about "Example files" that 
you want to search – except you aren't searching them, you're searching the 
metadata about them ... maybe?")

I had my own doubts about using whether "docs" as a "document type" would be 
confusing when i first started on this issue, but it wasn't until i went to 
change it that i realized just _how_ confusing it was, especially in the 
"search" examples, where {{"docs"}} is also the "response key" used for the 
SolrDocumentList, which then contained "docs" sub-documents ... eesh!
{quote}* In a section showing the child doc transformer, it used to have a link 
to further details at 
{{transforming-result-documents.adoc#child-childdoctransformerfactory}} but you 
changed it to {{other-parsers.adoc#block-join-children-query-parser}}. That 
doesn't make sense to me.
{quote}
i have no idea what this is in reference to? ... there are only 2 instances of 
"{{other-parsers.adoc#block-join-children-query-parser}}" and they are both in 
paragraphs discussing the "child" *parser*, not the transformer?
{noformat}
$ grep 'other-parsers.adoc#block-join-children-query-parser' src/*.adoc
src/searching-nested-documents.adoc:The `{!child}` query parser can be used to 
search for the _descendent_ documents of parent documents matching a wrapped 
query. For a detailed explanation of this parser, see the section 
<>.
src/searching-nested-documents.adoc:NOTE: The `of` local param is neccessary to 
tell the `{!child}` parser the set of _all_ "ancestor" documents to consider 
when looking for matching children.  In this example we've used `\*:* 
-\_nest_path_:*` to indicate we want to consider all documents which don't have 
a nest path field -- ie: all "root" level document.  When dealing with multiple 
levels of nested documents, it can be very tricky to define a correct `of` 
param -- see the 
<> for details.
{noformat}

> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> 

[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-29 Thread David Smiley (Jira)


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

David Smiley commented on SOLR-14383:
-

Hoss: please use PRs, which are much easier to peer review.  I've been dragging 
my feet, not looking forward to reading asciidoc diffs.  Kudos to Jan on 
pointing out GitHub's "rich diff" feature!
 * I really appreciate the conversion of plain JSON input docs to a real curl 
equivalent that I could use right away.
 * The bullet advising the ID to be a composite ID for all docs in the block 
really seems like it should be a TIP.
 * Typo: thir -> their
 * Typo: garuntee -> guarantee (multiple)
 * The example about office supplies uses a child doc label of "docs", seen in 
sending the data to Solr and in retrieval.  I find the use of this word 
confusing because it's a common word in Solr, so I suspect others might as 
well.  Perhaps "documentation" or "files" might be alternatives?  "files" is 
nice and short.
 * In a section showing the child doc transformer, it used to have a link to 
further details at 
{{transforming-result-documents.adoc#child-childdoctransformerfactory}} but you 
changed it to {{other-parsers.adoc#block-join-children-query-parser}}.  That 
doesn't make sense to me.
 
RE the nocommits relating to parent/child query parsers, perhaps [~mkhl] can 
help with that.  In my work with @moshebla on nested documents, we didn't touch 
them.  Ideally our project scope would have gone further and upgraded them too, 
like we did for the child doc transformer.  Alas; it was a partially 
implemented overhaul.
 

> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch, 
> SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-28 Thread Jira


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

Jan Høydahl commented on SOLR-14383:


{quote}
 NOTE: The `which` local param of the `\{!parent} parser serves the same purpse 
as the `\{!child}` parser's `of` param: to define the set of _all_ "parent" 
documents to consider for matching based on thier children. In this example 
we've again used `*:* -_nest_path_:*` to indicate we want to consider all 
documents which don't have a nest path field – ie: all "root" level document.
{quote}
This may be true in a simple parent/child relationship. But if you have 
parent/child/grandchild then, when selecting the grandchild level, the 
{{which}} part needs to define *all ancestors including both root documents and 
every intermediary parent level all the way down to the grandchild*. Else you 
get unexpected results. Please see this solr-user thread for details 
[https://lists.apache.org/thread.html/r681dc518118e316992b35f9e05aa88b7ad99424a8eca95d55f52eca6%40%3Csolr-user.lucene.apache.org%3E]
 

> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-28 Thread Jira


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

Jan Høydahl commented on SOLR-14383:


Typo in both the child and parent sections:
{quote}NOTE: The `of` local param is neccessary to tell the `\{!child} parser 
the set of _all_ "parent" documents to consider when looking for matching 
children. In this example we've used `*:* -_nest_path_:*` to indicate we want 
to consider all documents which don't have a nest path field – ie: all "root" 
level document.
{quote}
Need end back-quote after {{`\{!child}}} and {{`\{!parent}}}

> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-28 Thread Jira


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

Jan Høydahl commented on SOLR-14383:


Typo:

{quote}The same query with the addition of the {{[child]}} transformer is 
*showm* below{quote}

 

> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-28 Thread Jira


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

Jan Høydahl commented on SOLR-14383:


I made a temp branch with the patch here 
[https://github.com/apache/lucene-solr/compare/master...cominvent:14383-tmp] 
which makes it possible to click "Display the rich diff" on each file to see 
inline in Asciidoc render what changed.

> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch, SOLR-14383.patch, SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-07-22 Thread David Smiley (Jira)


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

David Smiley commented on SOLR-14383:
-

Hoss, I really appreciate you working on making the documentation here better!  
I took a look at the patch, applied it in IntelliJ and looked at the diff.  
Overall I have no major concerns.  You have some nocommits that I should 
respond to and maybe some other things.  I'm technically on vacation this week 
so maybe next week I can give more input.

> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Assignee: Chris M. Hostetter
>Priority: Major
> Attachments: SOLR-14383.patch
>
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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



[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear

2020-06-08 Thread Chris M. Hostetter (Jira)


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

Chris M. Hostetter commented on SOLR-14383:
---

i was recently reminded of this Jira when i tried to look up the current state 
of nested document support in solr 8.5, and realized a few additional things 
about these examples that are super confusing:
 # the field names used in these examples aren't valid in the default solr 
schema unless you have schemaless field guessing enabled – but even then the 
docs use "ID" (all caps) instead of "id" so trying to add these documents fails 
due to a missing uniqueKey
 # the examples only show the "raw" input documents, not how/where to index 
them – if you try to use 
{{[http://localhost:8983/solr/gettingstarted/update/json/docs]}} (the 
convenience path recommended in other places of the ref-guide because it 
automatically assumes the JSON content type and expects a list of JSON 
documents instead of a list of solr update commands) you will not get any 
nested docs, because that path has the implicit params for flattening custom 
json: 
[https://lucene.apache.org/solr/guide/8_5/transforming-and-indexing-custom-json.html]

> Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, 
> consistent, and clear
> --
>
> Key: SOLR-14383
> URL: https://issues.apache.org/jira/browse/SOLR-14383
> Project: Solr
>  Issue Type: Sub-task
>Reporter: Chris M. Hostetter
>Priority: Major
>
> As reported on solr-user@lucene by Peter Pimley...
> {noformat}
> The page "Indexing Nested Documents" has an XML example showing two
> different ways of adding nested documents:
> https://lucene.apache.org/solr/guide/8_5/indexing-nested-documents.html#xml-examples
> The text says:
>   "It illustrates two styles of adding child documents: the first is
> associated via a field "comment" (preferred), and the second is done
> in the classic way now referred to as an "anonymous" or "unlabelled"
> child document."
> However in the XML directly below there is no field named "comment".
> There is one named "content" and another named "comments" (plural),
> but no field named "comment".  In fact, looking at the Json example
> immediately below, I wonder if the XML element currently named
> "content" should be named "comments", and what is currently marked
> "comments" should be "content"?
> Secondly, in the Json example it says:
>   "The labelled relationship here is one child document but could have
> been wrapped in array brackets."
> However in the actual Json, the parent document (ID=1) with a labelled
> relationship has two child documents (IDs 2 and 3), and they are
> already in array brackets.
> {noformat}
> * The 2 examples (XML and JSON) should be updated to contains *structurally* 
> identical content, (ie: same number of documents, with same field values, and 
> same hierarchical relationships) to focus on demonstrating the syntax 
> differences (ie: things like the special {{\_childDocuments\_}} key in json)
> * The paragraphs describing the examples should be updated to:
> ** refer to the correct field names -- since both "comments" and "contents" 
> fields exist in the examples, it's impossible for novice users to even 
> udnerstand where th "typo" might be in the descriptions (I'm pretty 
> knowledgeable about Solr and even i'm second guessing myself as to what the 
> intent in these paragraphs are)
> ** refer to documents by {{"id"}} value, not just descriptors like "first" 
> and "second" 
> * it might be worth considering rewriting this section to use "callouts": 
> https://asciidoctor.org/docs/user-manual/#callouts -- similar to how we use 
> them in other sections like this: 
> https://lucene.apache.org/solr/guide/8_5/uploading-data-with-index-handlers.html#sending-json-update-commands



--
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