Details

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

      Description

      Currently, DocsEnum#freq() does not allow IOException. This is problematic if somebody wants to implement a codec that allows lazy loading of freq. Frequency will be read and decoded only when #freq() will be called, therefore calling IndexInput's read methods that can throw IOException.

      The current workaround is to catch the IOException in freq() and ignore it (which is not very nice and not a good solution).

      1. LUCENE-4046.patch
        14 kB
        Simon Willnauer

        Activity

        Hide
        Simon Willnauer added a comment -

        This makes perfect sense to me. While I don't see a very common usecase the entire codec stuff is more or less for uncommon usecases. Can you come up with a patch for this?

        Show
        Simon Willnauer added a comment - This makes perfect sense to me. While I don't see a very common usecase the entire codec stuff is more or less for uncommon usecases. Can you come up with a patch for this?
        Hide
        Renaud Delbru added a comment -

        Ok, I'll try to provide a patch in the coming weeks.

        Show
        Renaud Delbru added a comment - Ok, I'll try to provide a patch in the coming weeks.
        Hide
        Simon Willnauer added a comment -

        here is a patch adding IOException to DocsEnum#freq()

        Show
        Simon Willnauer added a comment - here is a patch adding IOException to DocsEnum#freq()
        Hide
        Simon Willnauer added a comment -

        committed to trunk

        Show
        Simon Willnauer added a comment - committed to trunk
        Hide
        Renaud Delbru added a comment -

        Great, Thanks Simon.

        Show
        Renaud Delbru added a comment - Great, Thanks Simon.
        Hide
        Simon Willnauer added a comment -

        Great, Thanks Simon.

        no worries! YW

        Show
        Simon Willnauer added a comment - Great, Thanks Simon. no worries! YW

          People

          • Assignee:
            Simon Willnauer
            Reporter:
            Renaud Delbru
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development