Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improvements to code involving synchronized maps #52

Open
wants to merge 15 commits into
base: trunk
Choose a base branch
from
Open

Improvements to code involving synchronized maps #52

wants to merge 15 commits into from

Commits on Oct 11, 2017

  1. Utilize enhanced for

    @dtrebbien
    dtrebbien committed Oct 11, 2017
    Configuration menu
    Copy the full SHA
    38fde81 View commit details
    Browse the repository at this point in the history

  2. Add a clarifying comment

    @dtrebbien
    dtrebbien committed Oct 11, 2017
    Configuration menu
    Copy the full SHA
    23a4761 View commit details
    Browse the repository at this point in the history

  3. Use standard WeakHashMap

    @dtrebbien
    dtrebbien committed Oct 11, 2017
    Configuration menu
    Copy the full SHA
    a2c5c9b View commit details
    Browse the repository at this point in the history

Commits on Oct 12, 2017

  1. Ensure that notifyUpdated() and notifyDestroyed() do not create too-s… 

    …mall arrays
    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    3b41d3f View commit details
    Browse the repository at this point in the history

  2. Fix a URL in a comment

    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    316689f View commit details
    Browse the repository at this point in the history

  3. Changes to org.apache.jackrabbit.jcr2spi.ItemImpl require a bump in m… 

    …ajor version number
    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    3a8cfc9 View commit details
    Browse the repository at this point in the history

  4. Make some local variables and loop variables final

    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    2d9bca8 View commit details
    Browse the repository at this point in the history

  5. Apply improvements from ItemImpl to PrivilegeRegistry

    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    ce450b2 View commit details
    Browse the repository at this point in the history

  6. Apply improvements from ItemImpl to NodeTypeRegistry

    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    cf7ee0d View commit details
    Browse the repository at this point in the history

  7. Need to synchronize on docNumber2id for both get() and put()

    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    0a8bb4a View commit details
    Browse the repository at this point in the history

  8. Apply improvements from ItemImpl to NodeTypeRegistryImpl

    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    67a1355 View commit details
    Browse the repository at this point in the history

  9. Fix potential non-determinism in DbDataStore.deleteAllOlderThan()

    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    8ce7d86 View commit details
    Browse the repository at this point in the history

  10. Make loop variable final

    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    648cd32 View commit details
    Browse the repository at this point in the history

  11. Improve the wording of some comments

    @dtrebbien
    dtrebbien committed Oct 12, 2017
    Configuration menu
    Copy the full SHA
    adaf2c0 View commit details
    Browse the repository at this point in the history

Commits on Oct 14, 2017

  1. Apply improvements from ItemImpl to CacheManager 

    This also fixes potential non-determinism in resizeAll() because all
access to the caches map must be synchronized, but the caches.size()
call wasn't synchronized.
    @dtrebbien
    dtrebbien committed Oct 14, 2017
    Configuration menu
    Copy the full SHA
    982cc0e View commit details
    Browse the repository at this point in the history