IllegalArgumentException with Play's auto-reload feature

Hi guys!

I am working on project based on Play framework. I integrated JOOQ with SimpleFlatMapper.

And there is a problem when auto-reload feature is enabled. Just after application start SimpleFlatMapper is working properly but after any changes to mapper creation chain I am getting this exception:

java.lang.IllegalArgumentException: object is not an instance of declaring class
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.simpleflatmapper.reflect.setter.MethodSetter.set(MethodSetter.java:17)
	at org.simpleflatmapper.map.fieldmapper.FieldMapperImpl.mapTo(FieldMapperImpl.java:24)
	at org.simpleflatmapper.map.generated.models.AsmMapperFromResultSetToSomeDTOInj8_I7.mapFields(Unknown Source)
	at org.simpleflatmapper.map.generated.models.AsmMapperFromResultSetToSomeDTOInj8_I7.mapFields(Unknown Source)
	at org.simpleflatmapper.map.mapper.AbstractMapper.map(AbstractMapper.java:27)
	at org.simpleflatmapper.map.impl.StaticMapperEnumarable.currentValue(StaticMapperEnumarable.java:30)

After application restart new code starts work correctly.

It happens for example when I add or remove addKeys() call

JdbcMapperFactory.newInstance()
        .propertyNameMatcherFactory(DefaultPropertyNameMatcherFactory.DEFAULT)
        .addKeys("id")
        .newMapper(SomeDTO.class).stream(resultSet).collect(Collectors.toList());

Thank you, Aleksey

[arnaudroger]

so was able to reproduce it, and figure out the problem, the asm factory failed to generated the asm setter because for some reason - my own stupidity - it try to generate a primitive setter for Integer. so then we have a mix of asm for instantiation and reflection for setter. The asm generation handles the class change with no problem but the reflection points to a method of the previous version.

If you disable asm then you have reflection all the way through so the v1 class is generated. I fixed the asm setter generation so we can have asm all the way too, which work as well - that explains why I could not reproduce the problem with String too.

I don’t know if I’ll be able to release in the next 2 days, might be later this week.

[AlekseyMko]

Looks like it’s working correctly now Thank you for the fix and for the explanation about grouping