[jira] [Commented] (BEANUTILS-340) Property with getter from generic interface returns wrong readMethod/propertyType on Linux environment

["Eric Westfall (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/BEANUTILS-340?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13433818#comment-13433818 ] 

Eric Westfall commented on BEANUTILS-340:
-----------------------------------------

Considering there hasn't been any activity on this jira and Java 6 is EOL, what I'm about to say may not end up being too important. But if someone does have a chance to try and integrate this fix in, I think you will want to review the attached patch. We actually experienced firsthand that it can be disastrous under load. It manifests in such a way that you start getting random errors about getters not being found on objects which obviously have those getters. The general stack trace looks like this:

{code}
java.lang.NoSuchMethodException: Property 'blah' has no getter method in class 'class my.package.MyClass'
	at org.apache.commons.beanutils.PropertyUtilsBean.getSimpleProperty(PropertyUtilsBean.java:1340)
	at org.apache.commons.beanutils.PropertyUtilsBean.getNestedProperty(PropertyUtilsBean.java:752)
        ...
{code}

The main problem appears to be in these two lines:

{code}
descriptor.setReadMethod(null);
descriptor.setWriteMethod(null);
{code}

First of all, this appears to be modifying the state in an unsafe way of PropertyDescriptors that are cached and used by multiple threads. This is likely the source of the problem because what we are seeing is situations where we get back a PropertyDescriptor with a null readMethod in random cases and only under load.

Our "fix" was to remove these two lines where it sets the method to null because they really seemed unnecessary. While this appeared to work around the issue, I still think it's likely that there is some unsafe stuff going on here from a concurrency perspective since it's still modifying property descriptor read/write methods, but it should happen in far fewer cases if those lines above are omitted.
                
> Property with getter from generic interface returns wrong readMethod/propertyType on Linux environment
> ------------------------------------------------------------------------------------------------------
>
>                 Key: BEANUTILS-340
>                 URL: https://issues.apache.org/jira/browse/BEANUTILS-340
>             Project: Commons BeanUtils
>          Issue Type: Bug
>          Components: Bean / Property Utils
>    Affects Versions: 1.8.0-BETA
>         Environment: ==Works correctly in==
> Windows XP
> java version "1.5.0_12"
> Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0_12-b04)
> Java HotSpot(TM) Client VM (build 1.5.0_12-b04, mixed mode)
> ==Fails in==
> Linux 2.6.27-gentoo-r8 #6 SMP Thu Feb 5 19:18:16 MST 2009 i686 06/17 GenuineIntel GNU/Linux
> java version "1.5.0_17"
> Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0_17-b04)
> Java HotSpot(TM) Server VM (build 1.5.0_17-b04, mixed mode)
>            Reporter: Dave Lindquist
>            Priority: Minor
>             Fix For: LATER THAN 1.8.4
>
>         Attachments: jira_340_new.patch, jira_340.patch
>
>
> PropertyUtils.getPropertyDescriptors is returning the wrong readMethod (and thus the wrong property type) when a method is implemented from a genericized interface, but only on some environments.  This seems to work on Windows, but fails on Linux.  (Compile environment does not matter, runtime environment does seem to matter.)
> Take the following test class:
> {code}
> public class Testing
> {
> 	public static void main(String[] args) throws Exception
> 	{
> 		for(PropertyDescriptor desc : PropertyUtils.getPropertyDescriptors(Test2.class))
> 		{
> 			if(desc.getName().equals("something"))
> 			{
> 				System.out.println(desc.getName() + "\t" + desc.getPropertyType() + "\t" + desc.getReadMethod() + "\t" + desc.getReadMethod().isSynthetic() + "\t" + desc.getReadMethod().isBridge());
> 			}
> 		}
> 	}
> 	
> 	// An interface with generics, and with getter and setter defined 'generically'.
> 	public static interface Test<T extends Number>
> 	{
> 		public T getSomething();
> 		
> 		public void setSomething(T something);
> 	}
> 	
> 	// A concrete class using a specific genericization of the interface (Long), with getter and setter implemented appropriately.
> 	public static class Test2 implements Test<Long>
> 	{
> 		public Long getSomething()
> 		{
> 			return(null);
> 		}
> 		
> 		public void setSomething(Long something)
> 		{
> 			
> 		}
> 	}
> }
> {code}
> When run on Windows XP, and working correctly, this prints:
>     something	class java.lang.Long	public java.lang.Long Testing$Test2.getSomething()	false	false
> indicating that it got the 'long' version of the method, and that this method is NOT synthetic or a bridge method.
> However, when run on Linux, this prints:
> something	class java.lang.Number	public volatile java.lang.Number Testing$Test2.getSomething()	true	true
> which is the signature from the interface, and is marked with both synthetic and bridge, indicating that this is not the 'real' method, but the compiler-created method due to generics.
> I think that it should be ignoring the 'synthetic/bridge' method auto-created by the compiler, but I'm not sure why it is environment-dependent.  Perhaps the environment somehow controls the method definition order?  (At runtime, not compile time, obviously.)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira