svn commit: r1800484 - in /ofbiz/ofbiz-plugins/trunk/solr: config/solrconfig.properties src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java

Author: jleroux
Date: Sat Jul  1 09:05:56 2017
New Revision: 1800484

URL: http://svn.apache.org/viewvc?rev=1800484&view=rev
Log:
Improved: Missing Java Option solr.log.dir. Logging may be missing or incomplete
(OFBIZ-9375)

This fixes and completes r1800457
I put a wrong path in solrconfig.properties for solr.log.dir
(/runtime/indexes/solr instead of runtime/logs/solr).
This also adds the log level.

Not sure this has a real impact on Solr logging (from the
Solr admin UI does not seem to work.) But at least we no longer have this
annoying message. People interested in Solr logging will dig in...

Modified:
    ofbiz/ofbiz-plugins/trunk/solr/config/solrconfig.properties
    ofbiz/ofbiz-plugins/trunk/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java

Modified: ofbiz/ofbiz-plugins/trunk/solr/config/solrconfig.properties
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-plugins/trunk/solr/config/solrconfig.properties?rev=1800484&r1=1800483&r2=1800484&view=diff
==============================================================================
--- ofbiz/ofbiz-plugins/trunk/solr/config/solrconfig.properties (original)
+++ ofbiz/ofbiz-plugins/trunk/solr/config/solrconfig.properties Sat Jul  1 09:05:56 2017
@@ -47,4 +47,6 @@ solr.client.connection.timeout=
 solr.client.trust.selfsigned.cert=true
 
 # Defines Solr specific log directory
-solr.log.dir=/runtime/indexes/solr
+solr.log.dir=runtime/logs/solr
+# Defines Solr log level
+solr.log.level=INFO

Modified: ofbiz/ofbiz-plugins/trunk/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-plugins/trunk/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java?rev=1800484&r1=1800483&r2=1800484&view=diff
==============================================================================
--- ofbiz/ofbiz-plugins/trunk/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java (original)
+++ ofbiz/ofbiz-plugins/trunk/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java Sat Jul  1 09:05:56 2017
@@ -67,6 +67,7 @@ public class OFBizSolrContextFilter exte
     public void init(FilterConfig config) throws ServletException {
         Properties props = System.getProperties();
         props.setProperty("solr.log.dir", UtilProperties.getPropertyValue("solrconfig", "solr.log.dir", "runtime/logs/solr"));
+        props.setProperty("solr.log.level", UtilProperties.getPropertyValue("solrconfig", "solr.log.level", "INFO"));
         super.init(config);
     }