myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Leonardo Uribe (JIRA)" <>
Subject [jira] Resolved: (MYFACES-2805) Use synchronized blocks for ApplicationImpl _defaultValidatorsIds
Date Fri, 09 Jul 2010 23:28:50 GMT


Leonardo Uribe resolved MYFACES-2805.

    Fix Version/s: 2.0.1-SNAPSHOT
       Resolution: Fixed

> Use synchronized blocks for ApplicationImpl _defaultValidatorsIds
> -----------------------------------------------------------------
>                 Key: MYFACES-2805
>                 URL:
>             Project: MyFaces Core
>          Issue Type: Improvement
>          Components: JSR-314
>    Affects Versions: 2.0.0
>            Reporter: Leonardo Uribe
>            Assignee: Leonardo Uribe
>             Fix For: 2.0.1-SNAPSHOT
> 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.

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

View raw message