Details

    • Type: Bug
    • 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

      We removed this for two reasons:

      • nothing was using it
      • the map is "unsafe" if a codec tried to write to it during in-place dv update.

      But Adrien has a real use case (LUCENE-5914), and I think we can just add some safety for the updates case (e.g. if the map is unmodifiable then the trap will not exist, any put() will throw exception). In general, we should have more safety in SI anyway (diagnostics map, too).

        Activity

        Hide
        anshumg Anshum Gupta added a comment -

        Bulk close after 5.0 release.

        Show
        anshumg Anshum Gupta added a comment - Bulk close after 5.0 release.
        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit 1642943 from Robert Muir in branch 'dev/branches/branch_5x'
        [ https://svn.apache.org/r1642943 ]

        LUCENE-6085: add back SI.attributes

        Show
        jira-bot ASF subversion and git services added a comment - Commit 1642943 from Robert Muir in branch 'dev/branches/branch_5x' [ https://svn.apache.org/r1642943 ] LUCENE-6085 : add back SI.attributes
        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit 1642919 from Robert Muir in branch 'dev/trunk'
        [ https://svn.apache.org/r1642919 ]

        LUCENE-6085: add back SI.attributes

        Show
        jira-bot ASF subversion and git services added a comment - Commit 1642919 from Robert Muir in branch 'dev/trunk' [ https://svn.apache.org/r1642919 ] LUCENE-6085 : add back SI.attributes
        Hide
        jpountz Adrien Grand added a comment -

        +1

        Show
        jpountz Adrien Grand added a comment - +1
        Hide
        mikemccand Michael McCandless added a comment -

        +1

        Show
        mikemccand Michael McCandless added a comment - +1
        Hide
        rcmuir Robert Muir added a comment -

        Here's a initial patch.

        Show
        rcmuir Robert Muir added a comment - Here's a initial patch.

          People

          • Assignee:
            Unassigned
            Reporter:
            rcmuir Robert Muir
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development