Uploaded image for project: 'Solr Search Provider'
  1. Solr Search Provider
  2. MGNLEESOLR-162

SolrQueryBuilder#search() should print errors at the proper level

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Neutral
    • 6.1.6
    • 5.6
    • None
    • DevX 42
    • 0.05
    • Yes

    Description

      Currently the code only outputs search errors at a DEBUG level.

          public String search() {
              try {
                  log.debug("Client name [{}].", clientName);
                  log.debug("Solr query [{}].", solrQuery);
                  QueryRequest req = new QueryRequest(solrQuery);
                  SolrClient solrClient = magnoliaSolrBridge.getSolrClient(clientName);
                  NoOpResponseParser rawJsonResponseParser = new NoOpResponseParser(CommonParams.JSON);
                  req.setResponseParser(rawJsonResponseParser);
                  String response = (String) solrClient.request(req).get("response");
                  log.debug("Response: {}]", response);
                  return response;
              } catch (Exception e) {
                  log.debug("Query to solr failed. ", e);  // <-- Should be ERROR level
                  return null;
              }
      

      See SolrQueryBuilder line 773.

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              mdivilek Milan Divilek
              rgange Richard Gange
              DeveloperX
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                Work Started:

                Checklists

                  Bug DoR
                  Task DoD

                  Time Tracking

                    Estimated:
                    Original Estimate - Not Specified
                    Not Specified
                    Remaining:
                    Remaining Estimate - Not Specified
                    Not Specified
                    Logged:
                    Time Spent - 1h
                    1h