Lucene - Core
  1. Lucene - Core
  2. LUCENE-3816

FilteredDocIdSet does not handle a case where the inner set iterator is null

    Details

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

      Description

      DocIdSet#iterator is allowed to return null, when used in FilteredDocIdSet, if null is returned from the inner set, the FilteredDocIdSetIterator fails since it does not allow for nulls to be passed to it.

      The fix is simple, return null in FilteredDocIdSet in the iterator method is the iterator is null.

      1. LUCENE-3816.patch
        3 kB
        Uwe Schindler
      2. LUCENE-3816.patch
        0.9 kB
        Shay Banon

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        13h 42m 1 Uwe Schindler 22/Feb/12 13:39
        Resolved Resolved Closed Closed
        442d 21h 4m 1 Uwe Schindler 10/May/13 11:44
        Uwe Schindler made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Uwe Schindler made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Uwe Schindler added a comment -

        Committed trunk revision: 1292282
        Committed 3.x revision: 1292288

        Thanks Shay!

        Show
        Uwe Schindler added a comment - Committed trunk revision: 1292282 Committed 3.x revision: 1292288 Thanks Shay!
        Uwe Schindler made changes -
        Attachment LUCENE-3816.patch [ 12515587 ]
        Hide
        Uwe Schindler added a comment -

        Patch with a new test that verifies this works. Fails without Shay's patch.

        Will commit now.

        Show
        Uwe Schindler added a comment - Patch with a new test that verifies this works. Fails without Shay's patch. Will commit now.
        Uwe Schindler made changes -
        Fix Version/s 3.6 [ 12319070 ]
        Fix Version/s 4.0 [ 12314025 ]
        Hide
        Uwe Schindler added a comment -

        Tha nks for reporting this, I will check and commit trunk and 3.x branch!

        Show
        Uwe Schindler added a comment - Tha nks for reporting this, I will check and commit trunk and 3.x branch!
        Uwe Schindler made changes -
        Assignee Uwe Schindler [ thetaphi ]
        Shay Banon made changes -
        Field Original Value New Value
        Attachment LUCENE-3816.patch [ 12515509 ]
        Hide
        Shay Banon added a comment -

        Patch attached.

        Show
        Shay Banon added a comment - Patch attached.
        Shay Banon created issue -

          People

          • Assignee:
            Uwe Schindler
            Reporter:
            Shay Banon
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development