Lucene - Core
  1. Lucene - Core
  2. LUCENE-3024

If index has more than Integer.MAX_VALUE terms, seeking can it AIOOBE due to long/int overflow

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.3
    • Component/s: None
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      Tom hit a new long/int overflow case: http://markmail.org/thread/toyl2ujcl4suqvf3

      This is a regression, in 3.1, introduced with LUCENE-2075.

      Worse, our Test2BTerms failed to catch this, so I've fixed that test to show the failure.

      1. LUCENE-3024.patch
        11 kB
        Michael McCandless

        Activity

        Hide
        Michael McCandless added a comment -

        Patch.

        This differs a bit from the patch I posted to the dev list – there was also an assert that was illegally casting to int.

        Show
        Michael McCandless added a comment - Patch. This differs a bit from the patch I posted to the dev list – there was also an assert that was illegally casting to int.
        Hide
        Michael McCandless added a comment -

        Fixed in 3.2, 4.0.

        I'm leaving this open in case we ever release 3.1.1.

        Show
        Michael McCandless added a comment - Fixed in 3.2, 4.0. I'm leaving this open in case we ever release 3.1.1.
        Hide
        Robert Muir added a comment -

        bulk move 3.2 -> 3.3

        Show
        Robert Muir added a comment - bulk move 3.2 -> 3.3
        Hide
        Michael McCandless added a comment -

        Resolving; we can reopen if we ever want to backport to 3.1.1.

        Show
        Michael McCandless added a comment - Resolving; we can reopen if we ever want to backport to 3.1.1.
        Hide
        Robert Muir added a comment -

        bulk close for 3.3

        Show
        Robert Muir added a comment - bulk close for 3.3

          People

          • Assignee:
            Michael McCandless
            Reporter:
            Michael McCandless
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development