Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Doc: @Transactional is inherited [SPR-5322] #9995

Closed
spring-projects-issues opened this issue Nov 26, 2008 · 1 comment
Closed

Doc: @Transactional is inherited [SPR-5322] #9995

spring-projects-issues opened this issue Nov 26, 2008 · 1 comment
Assignees
Labels
type: documentation A documentation task
Milestone

Comments

@spring-projects-issues
Copy link
Collaborator

Eberhard Wolff opened SPR-5322 and commented

Paragraph 9.5.6 "Using @Transactional" says:

The fact that annotations are not inherited means that if you are using class-based proxies (proxy-target-class="true") or the weaving-based aspect (mode="aspectj") then the transaction settings will not be recognised by the proxying/weaving infrastructure and the object will not be wrapped in a transactional proxy (which would be decidedly bad).

This is not true. @Transactional is marked as @Inherited. So a subclass that inherits from a superclass marked as @Transactional is not distinguishable from a class that is marked as@Transaction itself.


Affects: 1.2 RC1, 1.2 RC2, 1.2 final, 1.2.1, 1.2.2, 1.2.3, 1.2.4, 1.2.5, 1.2.6, 1.2.7, 1.2.8, 2.0 M1, 2.0 M2, 2.0 M3, 2.0 M4, 2.0 M5, 2.0 RC1, 2.0 RC2, 2.0 RC3, 2.0 RC4, 2.0 final, 2.0.1, 2.0.2, 1.2.9, 2.0.3, 2.0.4, 2.0.5, 2.0.6, 2.0.7, 2.0.8, 2.0.9, 2.1 M1, 2.1 M2, 2.1 M3, 2.1 M4, 2.5 RC1, 2.5 RC2, 2.5 final, 2.5.1, 2.5.2, 2.5.3, 2.5.4, 2.5.5, 2.5.6

@spring-projects-issues
Copy link
Collaborator Author

Juergen Hoeller commented

That sentence in the docs isn't worded very clearly. However, the context of that section is using @Transactional on interface methods: Since Java annotations are indeed not inherited from interfaces, the statement is correct in that context.

I've reworded that sentence to make the interface context clearer.

Juergen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation A documentation task
Projects
None yet
Development

No branches or pull requests

2 participants