You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2017/07/01 09:05:56 UTC
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);
}