accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From phrocker <...@git.apache.org>
Subject [GitHub] accumulo pull request: ACCUMULO-4153: Update the getCodec method t...
Date Wed, 24 Feb 2016 21:07:19 GMT
GitHub user phrocker opened a pull request:

    https://github.com/apache/accumulo/pull/73

    ACCUMULO-4153: Update the getCodec method to no longer be synchronize…

    …d and to use an atomic reference.
    
    This is based only on JSTACKs across a running system. I've seen improvement as a result
with local testing and will test at a larger scale as time permits. I can provide performance
numbers if needed for the PR. 
    
    I'm basing this against 1.6 as that is where we will test it and use it. It should merge
cleanly up to 1.7 and 1.8. 
    
    This is a very simple change. I added a unit test ( the last of which shows that we won't
run into the problem of creating many )...though if it fails for you let me know! 

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

    $ git pull https://github.com/phrocker/accumulo-1 ACCUMULO4153

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

    https://github.com/apache/accumulo/pull/73.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 #73
    
----
commit 7c689dda9d626da18af7e0a706201b3f67a96643
Author: phrocker <marc.parisi@gmail.com>
Date:   2016-02-24T20:40:08Z

    ACCUMULO-4153: Update the getCodec method to no longer be synchronized and to use an atomic
reference.

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message