[jboss-dev] new scanning in AS trunk

[Jaikiran Pai]

Ales Justin wrote:
> The Embedded needs to set this system property
>
>       // TODO -- remove this once we have Javassist Reflect performance fixed
>       String tif = System.getProperty("org.jboss.reflect.spi.TypeInfoFactory");
>       if (tif == null)
>          System.setProperty("org.jboss.reflect.spi.TypeInfoFactory", "org.jboss.reflect.plugins.introspection.IntrospectionTypeInfoFactory");
>
Looking at that TODO it looks temporary and a very specific config param 
to jboss.reflect. If a system property is required, then maybe it could 
be handled/added within the jboss-reflect project which knows the 
details about what this config is for? That way, numerous dependent 
projects need not change/add the param.


regards,
-Jaikiran
> Where do you do this for Embedded?
>
> btw, thanks for the info Shelly.
>
> -Ales
>
> On Jun 2, 2010, at 5:14 PM, Shelly McGowan wrote:
>
>> Ales,
>>
>> The Embedded tests run in the AS Build are failing with this change:
>>
>> http://hudson.jboss.org/hudson/view/JBoss%20AS/job/JBoss-AS-6.0.x/1646/console
>>
>> I've attached the output of the test.
>>
>> To reproduce, build jbossas/trunk passing -Pembedded
>>
>> Shelly McGowan
>> JBoss, by Red Hat
>>
>> ----- Original Message -----
>> From: "Ales Justin" <ales.justin at gmail.com>
>> To: "JBoss.org development list" <jboss-development at lists.jboss.org>
>> Cc: "Pete Muir" <pmuir at redhat.com>, "Remy Maucherat" <rmaucher at redhat.com>
>> Sent: Wednesday, June 2, 2010 9:26:32 AM GMT -05:00 US/Canada Eastern
>> Subject: [jboss-dev] new scanning in AS trunk
>>
>> I've just updated the AS trunk with new Scanning lib.
>> It also includes some other MC changes: Reflect, MDR, Kernel, CL, Deployers.
>>
>> Apart from AnnotationRepository (which was already there with the old code),
>> I've also hacked around Hibernate's Scanner a bit, so it uses new scanning lib's ScannerImpl.
>> (the hack is mostly in place due to some impl issues in Hibernate itself, Emmanuel is working on it)
>>
>> JSF can now use new JBossAnnotationProvider -- let me know how that goes Stan.
>> And web can use ResourcesIndex from DeploymentUnit -- Remy.
>>
>> WeldScanningPlugin is currently commented out.
>> We need to disable one of the deployers in plugin's favor.
>> Pete, let me know when the new Weld release is coming in,
>> and I'll enable the plugin + run a few tests.
>>
>> All in all, if you spot any issues / regressions wrt this change,
>> let me know, and I'll try to fix the stuff asap.
>>
>>
>> _______________________________________________
>> jboss-development mailing list
>> jboss-development at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/jboss-development
>> <org.jboss.jbossas.embedded.testsuite.ServerIntegrationTest-output.txt>_______________________________________________
>> jboss-development mailing list
>> jboss-development at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/jboss-development
>
>
> _______________________________________________
> jboss-development mailing list
> jboss-development at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-development