GitHub user spmason opened a pull request:

    https://github.com/apache/lucene-solr/pull/183

    LUCENE-7778 Removed synchronized from RAMFile methods

    These methods don't seem to benefit from locking - the methods that mutate 
the underlying fields aren't synchronized themselves, for example
    
    Removing these methods gives me a 2x throughput increase under concurrent 
load on internal benchmarks

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/spmason/lucene-solr-1 
ram-directory-remove-synchronized

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/lucene-solr/pull/183.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #183
    
----
commit 2439b917a33b91892e326ce4467cace46ad9ded5
Author: Steve Mason <[email protected]>
Date:   2017-04-11T11:08:38Z

    LUCENE-7778 Removed synchronized from RAMFile methods
    
    These methods don't seem to benefit from locking - the methods that mutate 
the underlying fields aren't synchronized themselves, for example

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to