Re: [jboss-dev] new scanning in AS trunk
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/co...
>
> 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(a)gmail.com>
> To: "JBoss.org development list" <jboss-development(a)lists.jboss.org>
> Cc: "Pete Muir" <pmuir(a)redhat.com>, "Remy Maucherat"
<rmaucher(a)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(a)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(a)lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-development
_______________________________________________
jboss-development mailing list
jboss-development(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-development