Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Lucene: Java-Dev

[jira] Updated: (LUCENE-2258) Remove "synchonized" from FuzzyTermEnum#similarity(final String target)

 

 

Lucene java-dev RSS feed   Index | Next | Previous | View Threaded


jira at apache

Feb 10, 2010, 12:22 AM

Post #1 of 1 (245 views)
Permalink
[jira] Updated: (LUCENE-2258) Remove "synchonized" from FuzzyTermEnum#similarity(final String target)

[ https://issues.apache.org/jira/browse/LUCENE-2258?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Uwe Schindler updated LUCENE-2258:
----------------------------------

Attachment: LUCENE-2258.patch

Patch.

> Remove "synchonized" from FuzzyTermEnum#similarity(final String target)
> -----------------------------------------------------------------------
>
> Key: LUCENE-2258
> URL: https://issues.apache.org/jira/browse/LUCENE-2258
> Project: Lucene - Java
> Issue Type: Improvement
> Components: Search
> Reporter: Uwe Schindler
> Assignee: Uwe Schindler
> Priority: Trivial
> Fix For: 2.9.2, Flex Branch, 3.0.1, 3.1
>
> Attachments: LUCENE-2258.patch
>
>
> The similarity method in FuzzyTermEnum is synchronized which is stupid because of:
> - TermEnums are the iterator pattern and so are single-thread per definition
> - The method is private, so nobody could ever create a fake FuzzyTermEnum just to have this method and use it multithreaded.
> - The method is not static and has no static fields - so instances do not affect each other
> The root of this comes from LUCENE-296, but was never reviewd and simply committed. The argument for making it synchronized is wrong.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscribe [at] lucene
For additional commands, e-mail: java-dev-help [at] lucene

Lucene java-dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.