Uploaded image for project: 'MyFaces Core'
  1. MyFaces Core
  2. MYFACES-2805

Use synchronized blocks for ApplicationImpl _defaultValidatorsIds

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.0.0
    • 2.0.1
    • JSR-314
    • None

    Description

      Right now we have a ConcurrentHashMap for ApplicationImpl _defaultValidatorsIds, but we are using a _cachedDefaultValidatorsIds too. In this case, it is better to use a synchronized block when default validators are added, but use a double-checked locking using volatile on the cache. Not use a synchronized block causes that getDefaultValidatorInfo() could return not all validators, but note that posibility is unlikely.

      Attachments

        Activity

          People

            lu4242 Leonardo Uribe
            lu4242 Leonardo Uribe
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: