Changeset 6670

Timestamp:

10/13/15 07:59:59 (9 years ago)

Author:

Twan Goosen

Message:

Don't use 'fast' request handler for facet queries, these interact with the user query which can cause parsing errors

Location:

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

Files:

• SolrDaoImpl.java (modified) (2 diffs)
• SolrFacetQueryFactoryImpl.java (modified) (1 diff)

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

@@ -11 +11 @@
 import org.apache.solr.common.SolrDocument;
 import org.apache.solr.common.SolrDocumentList;
+import org.apache.solr.common.SolrException;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -83 +84 @@
             logger.trace("Response: {}", response);
             return response;
+        } catch(SolrException e) {
+            logger.error("Error getting data:", e);
+            throw new RuntimeException(e);
         } catch (SolrServerException e) {
             logger.error("Error getting data:", e);

vlo/branches/vlo-3.3/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/service/solr/impl/SolrFacetQueryFactoryImpl.java

@@ -47 +47 @@
     public SolrQuery createFacetQuery(QueryFacetsSelection queryFacetsSelections, List<String> facets, int facetValueLimit) {
         final SolrQuery query = getBaseQuery(facets);
-        // we can use the 'fast' request handler here, document ranking is of no interest
-        query.setRequestHandler(FacetConstants.SOLR_REQUEST_HANDLER_FAST);
         addQueryFacetParameters(query, queryFacetsSelections);
         query.setFacetLimit(facetValueLimit);