[ 
https://issues.apache.org/jira/browse/MYFACES-2621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12848914#action_12848914
 ] 

Leonardo Uribe commented on MYFACES-2621:
-----------------------------------------

I think we don't really need synchronize methods on ExternalSpecifications. The 
reason is the variables:

    private static Boolean beanValidationAvailable;
    private static Boolean unifiedELAvailable;

only changes from null to Boolean.TRUE or from null to Boolean.FALSE, and after 
they are initialized, it never changes. So is multiple threads set this 
variable at the same time, everyone will set it with the same value (true or 
false, but never null). Ok, if we want to prevent that why don't use:

    private static volatile Boolean beanValidationAvailable;
    private static volatile Boolean unifiedELAvailable;

that is "one time safe publication" pattern.

> BeanValidation does not work with Unified EL 2.2
> ------------------------------------------------
>
>                 Key: MYFACES-2621
>                 URL: https://issues.apache.org/jira/browse/MYFACES-2621
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: JSR-314
>    Affects Versions: 2.0.0-beta-3
>         Environment: bean validation 1.0.0.GA, unified el 2.2
>            Reporter: Jakob Korherr
>            Assignee: Jakob Korherr
>
> When using the new Unified EL 2.2, BeanValidation does not work, because 
> _BeanValidatorUELUtils.getUELValueReferenceWrapper() always returns null.
> See also the related thread on the mailing list: 
> http://www.mail-archive.com/[email protected]/msg55250.html

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

Reply via email to