Solr
  1. Solr
  2. SOLR-3518

No `hits` in SolrResp. NamedList if distrib=true

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.0-ALPHA
    • Fix Version/s: 4.0
    • Component/s: SolrCloud
    • Labels:
      None
    • Environment:

      5.0-SNAPSHOT 1346798 - markus - 2012-06-06 11:38:15

      Description

      The hits field in the NamedList obtained from SolrQueryResponse.toLog() is not available for distrib=true requests. The hits fields is also not written to the log.

      See also:: http://lucene.472066.n3.nabble.com/SolrDispatchFilter-no-hits-in-response-NamedList-if-distrib-true-td3987751.html

        Activity

        Hide
        Markus Jelsma added a comment -

        Patch for trunk adding the `hits` field to the SolrQueryResponse's NamedList. It's only returned in the final response, not in intermediate shardrequests in a distributed search.

        Most likely not a good solution but it seems to work fine for now. Please improve.

        Show
        Markus Jelsma added a comment - Patch for trunk adding the `hits` field to the SolrQueryResponse's NamedList. It's only returned in the final response, not in intermediate shardrequests in a distributed search. Most likely not a good solution but it seems to work fine for now. Please improve.
        Hide
        Hoss Man added a comment -

        bulk fixing the version info for 4.0-ALPHA and 4.0 all affected issues have "hoss20120711-bulk-40-change" in comment

        Show
        Hoss Man added a comment - bulk fixing the version info for 4.0-ALPHA and 4.0 all affected issues have "hoss20120711-bulk-40-change" in comment
        Hide
        Robert Muir added a comment -

        rmuir20120906-bulk-40-change

        Show
        Robert Muir added a comment - rmuir20120906-bulk-40-change
        Hide
        Hoss Man added a comment -

        Most likely not a good solution but it seems to work fine for now. Please improve.

        It looks fine to me – there may still be a code path that are being overlooked (possibly with distributed grouping?) but it's certainly better then what we've got now.

        Committed revision 1382209. - trunk
        Committed revision 1382215. - 4x

        Thanks Markus!

        Show
        Hoss Man added a comment - Most likely not a good solution but it seems to work fine for now. Please improve. It looks fine to me – there may still be a code path that are being overlooked (possibly with distributed grouping?) but it's certainly better then what we've got now. Committed revision 1382209. - trunk Committed revision 1382215. - 4x Thanks Markus!
        Hide
        Commit Tag Bot added a comment -

        [branch_4x commit] Chris M. Hostetter
        http://svn.apache.org/viewvc?view=revision&revision=1382215

        SOLR-3518: Include final 'hits' in log information when aggregating a distibuted request (merge r1382209)

        Show
        Commit Tag Bot added a comment - [branch_4x commit] Chris M. Hostetter http://svn.apache.org/viewvc?view=revision&revision=1382215 SOLR-3518 : Include final 'hits' in log information when aggregating a distibuted request (merge r1382209)
        Hide
        Uwe Schindler added a comment -

        Closed after release.

        Show
        Uwe Schindler added a comment - Closed after release.

          People

          • Assignee:
            Hoss Man
            Reporter:
            Markus Jelsma
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development