Details

    • Type: Bug
    • Status: CLOSED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Scala 2.8.0, Scala 2.9.2
    • Fix Version/s: Scala 2.10.0-M3
    • Component/s: None
    • Labels:

      Description

      initialSize in the HashTable trait isn't used at all. Instead, the initialSize function of the companion object is used. This makes it pointless to override this function and provide a reasonable initialSize for loading a known number of entries. This results in very bad performance when loading a hash table with many thousand entries because the array gets continuously resized until it finally fits the number of entries.

        Attachments

          Activity

          Hide
          christian_schlichtherle Christian Schlichtherle added a comment -

          Any chance this will get ported back to 2.9.3? I mean a bug is a bug is a bug, isn't it?

          Show
          christian_schlichtherle Christian Schlichtherle added a comment - Any chance this will get ported back to 2.9.3? I mean a bug is a bug is a bug, isn't it?
          Hide
          moors Adriaan Moors added a comment - - edited

          fixed for 2.10 in f51dbd5564 – backport (by Typesafe) to 2.9.3-rc2 pending subject to available time

          Show
          moors Adriaan Moors added a comment - - edited fixed for 2.10 in f51dbd5564 – backport (by Typesafe) to 2.9.3-rc2 pending subject to available time
          Hide
          moors Adriaan Moors added a comment -

          reopened for potential backport

          Show
          moors Adriaan Moors added a comment - reopened for potential backport
          Hide
          moors Adriaan Moors added a comment -

          also pushing to 2.9.4-RC1

          Show
          moors Adriaan Moors added a comment - also pushing to 2.9.4-RC1
          Hide
          retronym Jason Zaugg added a comment -

          The backport isn't forthcoming, closing this ticket.

          Show
          retronym Jason Zaugg added a comment - The backport isn't forthcoming, closing this ticket.

            People

            • Assignee:
              prokopec Aleksandar Prokopec
              Reporter:
              christian_schlichtherle Christian Schlichtherle
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: