Changeset 4527

Timestamp:

02/17/14 16:00:09 (10 years ago)

Author:

twagoo

Message:

Added some logging to SOLR DAO's

Location:

vlo/branches/vlo-3.0/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/service/impl

Files:

• SearchResultsDaoImpl.java (modified) (2 diffs)
• SolrDaoImpl.java (modified) (3 diffs)

vlo/branches/vlo-3.0/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/service/impl/SearchResultsDaoImpl.java

@@ -10 +10 @@
 import org.apache.solr.client.solrj.response.QueryResponse;
 import org.apache.solr.common.SolrDocumentList;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class SearchResultsDaoImpl extends SolrDaoImpl implements SearchResultsDao {
+
+    private final static Logger logger = LoggerFactory.getLogger(SearchResultsDaoImpl.class);
 
     public SearchResultsDaoImpl(SolrServer solrServer, VloConfig config) {
@@ -27 +31 @@
     @Override
     public List<FacetField> getFacets(SolrQuery query) {
-        QueryResponse response = fireQuery(query);
-        return response.getFacetFields();
+        final QueryResponse response = fireQuery(query);
+        final List<FacetField> facetFields = response.getFacetFields();
+        logger.debug("Found {} facet fields", facetFields.size());
+        return facetFields;
     }
 

vlo/branches/vlo-3.0/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/service/impl/SolrDaoImpl.java

@@ -17 +17 @@
 public class SolrDaoImpl {
     
-    private final static Logger LOG = LoggerFactory.getLogger(SolrDaoImpl.class);
+    private final static Logger logger = LoggerFactory.getLogger(SolrDaoImpl.class);
     private final SolrServer solrServer;
     private final VloConfig config;
@@ -83 +83 @@
         sanitisedQuery = sanitise(query);
         try {
-            return solrServer.query(sanitisedQuery);
+            logger.debug("Executing query: {}", query);
+            final QueryResponse response = solrServer.query(sanitisedQuery);
+            logger.debug("Response: {}", response);
+            return response;
         } catch (SolrServerException e) {
-            LOG.error("Error getting data:", e);
+            logger.error("Error getting data:", e);
             throw new RuntimeException(e);
         }
@@ -100 +103 @@
         SolrDocumentList docs = fireQuery(query).getResults();
         if (docs.getNumFound() > 1) {
-            LOG.error("Error: found multiple documents for id (will return first one): " + docId + " \nDocuments found: " + docs);
+            logger.error("Error: found multiple documents for id (will return first one): " + docId + " \nDocuments found: " + docs);
             result = docs.get(0);
         } else if (docs.getNumFound() == 1) {