Lucene - Core
  1. Lucene - Core
  2. LUCENE-2711

BooleanScorer.nextDoc should also delegate to sub-scorer's bulk scoring method

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.1, 4.0-ALPHA
    • Component/s: core/search
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      BooleanScorer uses the bulk score methods of its sub scorers, asking them to score each chunk of 2048 docs.

      However, its .nextDoc fails to do this, instead manually walking through the sub's docs (calling .nextDoc()), which is slower (though this'd be tiny in practice).

      As far as I can tell it should delegate to the bulk scorer just like it does in its bulk scorer method.

      1. LUCENE-2711.patch
        1 kB
        Michael McCandless

        Activity

        Michael McCandless created issue -
        Michael McCandless made changes -
        Field Original Value New Value
        Attachment LUCENE-2711.patch [ 12457330 ]
        Michael McCandless committed 1023711 (1 file)
        Reviews: none

        LUCENE-2711: also use bulk sub-scorer API in BooleanScorer.nextDoc

        Michael McCandless committed 1023723 (92 files)
        Reviews: none

        LUCENE-2711: also use bulk sub-scorer API in BooleanScorer.nextDoc

        Lucene branch_3x
        Michael McCandless made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Mark Thomas made changes -
        Workflow jira [ 12524416 ] Default workflow, editable Closed status [ 12563693 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12563693 ] jira [ 12585254 ]
        Hide
        Grant Ingersoll added a comment -

        Bulk close for 3.1

        Show
        Grant Ingersoll added a comment - Bulk close for 3.1
        Grant Ingersoll made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Michael McCandless
            Reporter:
            Michael McCandless
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development