Lucene - Core
  1. Lucene - Core
  2. LUCENE-6058

Changes to Beider-Morse Encoder in latest commons-codec 1.10 release

    Details

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

      Description

      We have just recently released commons-codec 1.10 and I wanted to inform you that there have been slight changes to the Beider Morse encoder.

      This might result in slightly different encodings (see CODEC-187). At least one unit test has to be adjusted when updating.

      The results of the Beider Morse encoder are now identical to the reference implementation of the algorithm (v3.4, http://stevemorse.org/census/soundex.html), it might be necessary to inform users before upgrading as they might have to re-index to get consistent results.

        Issue Links

          Activity

          Hide
          Michael McCandless added a comment -

          Thanks, I updated its unit test based on the changes. It's marked experimental so it's allowed to change between releases... but I'll updated the CHANGES.txt entry to advertise this.

          Show
          Michael McCandless added a comment - Thanks, I updated its unit test based on the changes. It's marked experimental so it's allowed to change between releases... but I'll updated the CHANGES.txt entry to advertise this.
          Hide
          ASF subversion and git services added a comment -

          Commit 1638331 from Michael McCandless in branch 'dev/trunk'
          [ https://svn.apache.org/r1638331 ]

          LUCENE-6058: explain that BeiderMorseFilter changed its behavior

          Show
          ASF subversion and git services added a comment - Commit 1638331 from Michael McCandless in branch 'dev/trunk' [ https://svn.apache.org/r1638331 ] LUCENE-6058 : explain that BeiderMorseFilter changed its behavior
          Hide
          ASF subversion and git services added a comment -

          Commit 1638332 from Michael McCandless in branch 'dev/branches/branch_5x'
          [ https://svn.apache.org/r1638332 ]

          LUCENE-6058: explain that BeiderMorseFilter changed its behavior

          Show
          ASF subversion and git services added a comment - Commit 1638332 from Michael McCandless in branch 'dev/branches/branch_5x' [ https://svn.apache.org/r1638332 ] LUCENE-6058 : explain that BeiderMorseFilter changed its behavior
          Hide
          ASF subversion and git services added a comment -

          Commit 1651496 from hossman@apache.org in branch 'dev/trunk'
          [ https://svn.apache.org/r1651496 ]

          LUCENE-6058: call out this change as a re-index requirement in Solr upgrade instructions

          Show
          ASF subversion and git services added a comment - Commit 1651496 from hossman@apache.org in branch 'dev/trunk' [ https://svn.apache.org/r1651496 ] LUCENE-6058 : call out this change as a re-index requirement in Solr upgrade instructions
          Hide
          ASF subversion and git services added a comment -

          Commit 1651502 from hossman@apache.org in branch 'dev/branches/branch_5x'
          [ https://svn.apache.org/r1651502 ]

          LUCENE-6058: call out this change as a re-index requirement in Solr upgrade instructions (merge r1651496)

          Show
          ASF subversion and git services added a comment - Commit 1651502 from hossman@apache.org in branch 'dev/branches/branch_5x' [ https://svn.apache.org/r1651502 ] LUCENE-6058 : call out this change as a re-index requirement in Solr upgrade instructions (merge r1651496)
          Hide
          Michael McCandless added a comment -

          Woops, thanks Hoss!

          Show
          Michael McCandless added a comment - Woops, thanks Hoss!
          Hide
          Anshum Gupta added a comment -

          Bulk close after 5.0 release.

          Show
          Anshum Gupta added a comment - Bulk close after 5.0 release.

            People

            • Assignee:
              Unassigned
              Reporter:
              Thomas Neidhart
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development