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

Version should not be encoded as a String in the index

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 5.0, 6.0
    • None
    • None
    • 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.patch
          6 kB
          Michael McCandless
        2. LUCENE-5992.patch
          8 kB
          Michael McCandless
        3. LUCENE-5992.patch
          3 kB
          Robert Muir
        4. LUCENE-5992.patch
          3 kB
          Robert Muir
        5. LUCENE-5992_tests.patch
          25 kB
          Robert Muir

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved: