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

clean up serialization in the codebase

        Details

        • Type: Task
        • Status: Closed
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 4.0-ALPHA
        • Component/s: None
        • Labels:
          None
        • Lucene Fields:
          New, Patch Available

          Description

          We removed contrib/remote, but forgot to cleanup serialization hell everywhere.

          this is no longer needed, never really worked (e.g. across versions), and slows
          development (e.g. i wasted a long time debugging stupid serialization of
          Similarity.idfExplain when trying to make a patch for the scoring system).

          1. Text File
            LUCENE-2908.patch
            101 kB
            Robert Muir

            Issue Links

            supercedes

            Bug - A problem which impairs or prevents the functions of the product. LUCENE-2707 NumericField throws a NotSerializableException when it is serialized, even though it implements Serializable

            • Major - Major loss of function.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              rcmuir Robert Muir added a comment -

              attached is a patch. all tests pass.

              Show
              rcmuir Robert Muir added a comment - attached is a patch. all tests pass.
              Hide
              Permalink
              simonw Simon Willnauer added a comment -

              big +1 to get rid of Serializable its broken anyway, slow and not really working across versions! Folks that want to send stuff through the wire using java serialization should put api sugar on top.

              Show
              simonw Simon Willnauer added a comment - big +1 to get rid of Serializable its broken anyway, slow and not really working across versions! Folks that want to send stuff through the wire using java serialization should put api sugar on top.
              Hide
              Permalink
              thetaphi Uwe Schindler added a comment -

              +1

              Show
              thetaphi Uwe Schindler added a comment - +1
              Hide
              Permalink
              mikemccand Michael McCandless added a comment -

              +1

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

              Committed revision 1068526.

              Show
              rcmuir Robert Muir added a comment - Committed revision 1068526.
              Hide
              Permalink
              earwin Earwin Burrfoot added a comment -

              Oh, damn
              On my project, we specifically use java-serialization to pass configured Queries/Filters between cluster nodes, as it saves us HEAPS of wrapping/unwrapping them into some parallel serializable classes.

              Show
              earwin Earwin Burrfoot added a comment - Oh, damn On my project, we specifically use java-serialization to pass configured Queries/Filters between cluster nodes, as it saves us HEAPS of wrapping/unwrapping them into some parallel serializable classes.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development