[jira] [Commented] (SOLR-14383) Fix indexing-nested-documents.adoc XML/JSON examples to be accurate, consistent, and clear
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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