Details

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

      Description

      We already have packed data structures that support more than 2B values such as AppendingLongBuffer and MonotonicAppendingLongBuffer but none of them supports random write-access.

      We could write a PagedGrowableWriter for this, which would essentially wrap an array of GrowableWriters.

      1. LUCENE-5026.patch
        21 kB
        Adrien Grand
      2. LUCENE-5026.patch
        12 kB
        Adrien Grand

        Activity

        Hide
        Adrien Grand added a comment -

        Patch. I didn't make PagedGrowableWriter used anywhere but I think Mike is planning to use it for FST.

        Show
        Adrien Grand added a comment - Patch. I didn't make PagedGrowableWriter used anywhere but I think Mike is planning to use it for FST.
        Hide
        Adrien Grand added a comment -

        New patch that:

        • doesn't over-allocate the last page as requested per Mike,
        • shares more code with BlockPackedWriter|Reader (regarding fixed-size block handling)

        Unless someone objects, I'd like to commit it tomorrow.

        Show
        Adrien Grand added a comment - New patch that: doesn't over-allocate the last page as requested per Mike, shares more code with BlockPackedWriter|Reader (regarding fixed-size block handling) Unless someone objects, I'd like to commit it tomorrow.
        Hide
        Michael McCandless added a comment -

        +1, thanks Adrien.

        I think you don't need that super() in PagedGrowableWriter's ctor.

        Show
        Michael McCandless added a comment - +1, thanks Adrien. I think you don't need that super() in PagedGrowableWriter's ctor.
        Hide
        Steve Rowe added a comment -

        Bulk close resolved 4.4 issues

        Show
        Steve Rowe added a comment - Bulk close resolved 4.4 issues

          People

          • Assignee:
            Adrien Grand
            Reporter:
            Adrien Grand
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development