Bug 55534

Summary: Tomcat 7 and 8 missing javax.annotation classes added in Java EE 6, 7 respectively
Product: Tomcat 7 Reporter: Nick Williams <nicholas>
Component: Servlet & JSP APIAssignee: Tomcat Developers Mailing List <dev>
Severity: normal    
Priority: P2    
Version: 7.0.42   
Target Milestone: ---   
Hardware: All   
OS: All   

Description Nick Williams 2013-09-07 23:00:47 UTC
JSR 250 ("Common Annotations 1.0") specified a set of common annotations for application development. Some of them were included in Java SE 6 and later, but many of them are Java EE-specific and were added in Java EE 5. Tomcat 6 included these annotations since Tomcat supported some of them. The annotations included in Tomcat 6 are:


This was fine in Java EE 5 / Tomcat 6. However, Java EE 6 ("Common Annotations 1.1") added the following annotations that Tomcat 7 never included:


Furthermore, Java EE 7 ("Common Annotations 1.2") added the following annotation that Tomcat 8 has not yet included:


Tomcat should included these annotations in its "annotations-api.jar". Other frameworks (like Spring Framework and JPA providers) make use of these new annotations. Since Tomcat does not include them, the /correct/ thing for an application that needs them to do is include the official javax.annotation-api.jar artifact in their application. However, this JAR redefines annotations already supplied by Tomcat, which could lead to hard-to-diagnose errors (similar to why an application should never include the servlet-api artifact in their application). Adding these missing annotations would remove the requirement for applications to include the problem JAR in their application.

Tomcat 7 should ship @ManagedBean, @DataSourceDefinition, and @DataSourceDefinitions. Tomcat 8 should ship with @Priority.
Comment 1 Mark Thomas 2013-09-09 11:21:41 UTC
A review also spotted some errors in the existing common annotations 1.0 implementation.

Note that some of the official Java EE 5 Javadoc for common annotations is not correct.

Everything has been fixed for 8.0.x and 7.0.x and will be included in 8.0.0-RC2 and 7.0.43.

The CA 1.0 fixes have been proposed for 6.0.x.
Comment 2 Nick Williams 2013-09-09 16:16:36 UTC
Excellent. Good news! Thanks for attacking this so quickly.