Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.1, 6.0
    • Component/s: core/codecs
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      Its confusing and unnecessary that we compute this from docFreq for the doc/freq vint count, but write it for the positions case: its totalTermFreq % BLOCK_SIZE.

      1. LUCENE-4497.patch
        4 kB
        Robert Muir
      2. LUCENE-4497.patch
        4 kB
        Robert Muir

        Activity

        Hide
        Robert Muir added a comment -

        updated patch to trunk. This is actually a nice little savings to the positions file with the luceneutil 1M collection.

        trunk: 116425749 bytes
        patch: 111340216 bytes

        Show
        Robert Muir added a comment - updated patch to trunk. This is actually a nice little savings to the positions file with the luceneutil 1M collection. trunk: 116425749 bytes patch: 111340216 bytes
        Hide
        Michael McCandless added a comment -

        +1, nice!

        Show
        Michael McCandless added a comment - +1, nice!
        Hide
        Robert Muir added a comment -

        I committed to trunk. will bake for a bit before backporting.

        Show
        Robert Muir added a comment - I committed to trunk. will bake for a bit before backporting.
        Hide
        Commit Tag Bot added a comment -

        [branch_4x commit] Robert Muir
        http://svn.apache.org/viewvc?view=revision&revision=1401161

        LUCENE-4497: don't write PosVIntCount in 4.1 codec

        Show
        Commit Tag Bot added a comment - [branch_4x commit] Robert Muir http://svn.apache.org/viewvc?view=revision&revision=1401161 LUCENE-4497 : don't write PosVIntCount in 4.1 codec

          People

          • Assignee:
            Unassigned
            Reporter:
            Robert Muir
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development