Overriding annotations
While annotations have been very well received by the community, I have repeatedly been asked at JavaOne about the possibility of overriding certain annotations later during the cycle (at deployment time).
Here are a few thoughts on this subject.
The overriding is specified in an external XML file which could look like this:
<override>
<annotation-class names="javax.ejb3.TransactionAttribute" />
<scope>
<package names="com.foo" />
<class names="Account" />
<method names="deposit withdraw" />
</scope>
<overridden-values>
<value name="value"
old-value="javax.ejb3.TransactionAttributeType.SUPPORTS" />
new-value="javax.ejb3.TransactionAttributeType.REQUIRED" />
</overridden-values>
</override>
An override is specified with three parameters:
- What annotation type is being overridden .
- Hani pointed out that he didn’t like space-separated names, which I tend to agree with.
- The syntax should allow for regular expressions, including the possibility to exclude certain patterns.
- The scope of the override (which Java elements are impacted: package, class, method, field, etc…).
- Which values are overridden and what the new values are.
Notice in the file above that “names” is plural, so you can specify several space-separated elements each time.
Anything under a scope is “and’ed” together. In the example above, only methods named “deposit()” and “withdraw()” inside the class com.foo.Account will be candidates for overriding. We could imagine having several
The
The matching of the values in the XML file to the actual enum type can be performed through reflection, including enums, since it is possible to retrieve a String version of each enum value.
The API could probably be very similar to the existing one:
AnnotationOverrider ao = new AnnotationOverrider("override.xml");
Method m = ...; // retrieve the method
TransactionType oa = ao.getAnnotation(m, TransactionType.class);
// oa.value() now contains the overridden annotation
Here are some open questions/concerns that have come up so far:
- We should be able to override defaults as well.
- Should we make it possible to insert annotations and not just override existing ones?
- Specifying compound values could be problematic.
- Is the syntax too verbose? For example, Bill Burke is suggesting to insert entire Java snippets to identify the targets:
<annotation method="public void setEM(javax.ejb.EntityManager)">
@javax.ejb.Inject
</annotation>
I personally think that writing Java code in an XML file is a very strong design smell, and it’s a step backward since we are reintroducing redundancy in the programming model (if you rename this method in your Java code, you need to remember to update the fragile string in your deployment descriptor).
Please let us know what you think.
Comments (8)
http://radio.javaranch.com/channel/val/2004/07/23/1090568800000.html
Cedric, Hey, just wanted to refute some of your comments. :) You wrote: "if you rename this method in your Java code, you need to remember to update the fragile string in your deployment descriptor" I don't see this problem going away with your suggestion. Your proposal still needs: a) The method name b) the Annotation name c) The annotation's member value name d) The annotation values themselves Also, your approach doesn't handle overriden methods. (void foo(int), void foo(double), etc...). I'll will blog about my proposal and post the link here.
Another comment I wanted to make is that your proposal is solely about overriding annotations. I think there absolutely positively needs to be the ability to define/add/insert annotation definitions in XML as well. The use case is in Entity beans where you will want to define column/table mappings on a per deployment basis. Same goes for IDGenerators, etc.. that may be different per vendor DBMS. Bill
Ok, I'm flooding you sorry... but here's my blog on this: http://jboss.org/jbossBlog/blog/bburke/?permalink=XML+overrides+for+EJB3.html
There's no problem in computer science that can't be solved by yet another level of indirection.
So we wanted to replace the complexity of having meta data in external XML files with annotations... now we need meta-meta-data in external XML files to describe the changes we'd like to make to our meta-data. Nice.
http://www.jroller.com/page/letiemble/20040724#overriding_annotations
Some feedback as promised: http://radio.javaranch.com/channel/val/2004/07/26/1090824307000.html ;)