jsr107/jsr107spec — Issue #35: IsolationLevel tweaks
Author: gregrluck
Created: 2011-11-09 02:23:57+00:00
State: open

1. Two of the isolation levels are named incorrectly:
- READ_REPEATABLE should instead be REPEATABLE_READ
- READ_SERIALIZABLE should instead be SERIALIZABLE

2. The enum constant names are redundantly prefixed with "TX_" when the enum itself is already in the javax.cache.transaction package. Precedent for this style of annotation naming can be seen in TimeUnit, Thread.State, Proxy.Type etc…

3. The method name getValue() is particularly confusing as the enum already has the methods values() and valueOf() that utilize a different definition of the term "value". I would prefer a name that identifies the significance of the returned integer without having to read the Javadoc. Something along the lines of getJavaSqlConstant() would be ideal.