Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-5992

Version should not be encoded as a String in the index

    Details

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

      Description

      The version is really "just" 3 (maybe 4) ints under-the-hood, but today we write it as a String which then requires spooky string tokenization/parsing when we open the index. I think it should be encoded directly as ints.

      In LUCENE-5952 I had tried to make this change, but it was controversial, and got booted.

      Then in LUCENE-5969, I tried again, but that issue has morphed (nicely!) into fixing all sorts of things except these three ints.

      Maybe 3rd time's a charm

        Attachments

        1. LUCENE-5992_tests.patch
          25 kB
          Robert Muir
        2. LUCENE-5992.patch
          3 kB
          Robert Muir
        3. LUCENE-5992.patch
          3 kB
          Robert Muir
        4. LUCENE-5992.patch
          8 kB
          Michael McCandless
        5. LUCENE-5992.patch
          6 kB
          Michael McCandless

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved: