[hibernate-dev] Hibernate.next version and AS 7.2 community release...
Steve Ebersole
steve at hibernate.org
Tue Jun 26 15:21:25 EDT 2012
Well I think one option would be to explore using Jandex within
commons-annotations. Jandex is meant as a very lightweight (read:
FAST) annotation reader. commons-annotations is meant to read
annotations and merge the annotated values with xml overrides. But
commons-annotations could certainly use Jandex for the "read
annotations" portion of its mandate (I think). I think
commons-annotations is slow also because of how it reads XML. Reading
XML is always going to be a balance between (relative) speed and
readability/maintainability. In the new ORM code, we moved to StAX +
JAXB. My preliminary testing showed that to be a pretty great
compromise on that balance scale. But commons-annotations does not at
all work with JAXB nor JAXP. As far as I know it just directly used
dom4j apis (granted most likely because that is what ORM used).
And for the record, I never said throw commons annotations away.
Stopping using it in a particular project moving forward is much
different. I just don't think it makes sense to write "throw away
code". I'd avoid fixing commons-annotations and retrofitting fixes
into the old org.hibernate.mapping package. We know that code is going
away. If someone wants to write such throw away code, be my guest :)
On Tue 26 Jun 2012 01:39:30 PM CDT, Steve Ebersole wrote:
> Agree 10,000%
>
>
> On Tue 26 Jun 2012 11:29:04 AM CDT, Hardy Ferentschik wrote:
>>
>> On Jun 26, 2012, at 6:19 PM, Sanne Grinovero wrote:
>>
>>> an optimization we have been discussing but had still no time to
>>> implement is to update hibernate-commons-annotations to use Jandex.
>>
>> +1 for Jandex. However, I don't think it is worth to trying to use it
>> with the old configuration code.
>> The new metamodel will use jandex and I rather see that we get this
>> work complete instead of
>> making major changes to the existing annotation processing. That's
>> kind of doing twice the work.
>>
>> -1 for updating hibernate-commons-annotations. I think we should get
>> rid of it.
>>
>> --Hardy
>>
>>
>> _______________________________________________
>> hibernate-dev mailing list
>> hibernate-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/hibernate-dev
--
steve at hibernate.org
http://hibernate.org
More information about the hibernate-dev
mailing list