dsmiley commented on a change in pull request #1200: SOLR-12045: emphasize 
deployment caveat.
URL: https://github.com/apache/lucene-solr/pull/1200#discussion_r369824330
 
 

 ##########
 File path: solr/solr-ref-guide/src/analytics.adoc
 ##########
 @@ -33,7 +33,7 @@ Since the Analytics framework is a _search component_, it 
must be declared as su
 For distributed analytics requests over cloud collections, the component uses 
the `AnalyticsHandler` strictly for inter-shard communication.
 The Analytics Handler should not be used by users to submit analytics requests.
 
-To use the Analytics Component, the first step is to install this contrib 
module's plugins into Solr -- see the 
<<solr-plugins.adoc#installing-plugins,Solr Plugins>> section on how to do this.
+To use the Analytics Component, the first step is to install this contrib 
module's plugins into Solr -- see the 
<<solr-plugins.adoc#installing-plugins,Solr Plugins>> section on how to do 
this. Note: method with `<lib>` directories doesn't work, also consider copying 
`solr-analytic-x.x.jar` to 
`${solr.install.dir}/server/solr-webapp/webapp/WEB-INF/lib/`. 
 
 Review comment:
   The way you worded this, it suggests it might be deployed to two places.  It 
should be one or the other.  The word "also" and "copy" both contributed to 
this misunderstanding.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to