[jboss-dev] new scanning in AS trunk

[Jaikiran Pai]

Embedded resides here 
http://anonsvn.jboss.org/repos/jbossas/projects/embedded/trunk but it's 
mainly backed by the bootstrap impl-as module which sets most of the 
system properties in JBossASServerInitializer 
http://anonsvn.jboss.org/repos/jbossas/projects/bootstrap/trunk/impl-as/src/main/java/org/jboss/bootstrap/impl/as/server/JBossASServerInitializer.java. 
So I _think_ that's where this would go. ALR would definitely know.

I haven't yet tried adding that property to bootstrap (although I can 
try it sometime later if no one else gets to it before me).

regards,
-Jaikiran
Ales Justin wrote:
> I'll revert to Introspection by default tomorrow.
> And I'll check why the value convert fails if running in Javassist mode.
>
> Meanwhile, can someone just add the same property to Embedded,
> and let me know where this is, so I can remove it after I update Reflect.
>
> -Ales
>
> On Jun 2, 2010, at 5:37 PM, Jaikiran Pai wrote:
>
>> 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
>> _______________________________________________
>> jboss-development mailing list
>> jboss-development at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/jboss-development
>