ctargett commented on a change in pull request #549: WIP:SOLR-13129 URL: https://github.com/apache/lucene-solr/pull/549#discussion_r254689349
########## File path: solr/solr-ref-guide/src/index.adoc ########## @@ -90,6 +90,27 @@ The *<<getting-started.adoc#getting-started,Getting Started>>* section guides yo **** -- +[.row.match-my-cols] Review comment: A single page is fine IMO, even though it does make finding the right place for it more difficult. I think it's good to cover the indexing and the searching together when they are so specifically connected, as in this case. If the search section can be improved with better context for the links that go out to other pages that cover the query parsers, then the bulk of the new content for the page would be about the data modelling needed to index nested documents, which makes a stronger argument for it being in the indexing section. But I'm not 100% on that opinion - if you guys would prefer it under Searching, that would be OK (someday I'd like to re-org the top-level topics, but that's another whole problem we can't solve here). ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org