In the HibernatePersistenceProviderAdaptor, there is an unnecessary declaration of JtaManager. The sole purpose of having this variable is only to use in the overridden method named InjectJtaManager.
This was be improved by removing the JtaManager as there is no use of the variable, as of now at least.
In a shared containerized environment like Travis, it's not obvious that
the second read will be faster than the first one.
And indeed, the test often fails.
There's no real value in testing that, we are just testing I/O, and the OS
cache.
- checkstyle fixups
- adjust checkstyle to allow up to 5 newlines at the end of the file (1 to 5)
- fixed bad 4.6 Gradle wrapper version (cant load builds)
- functional changes - building, all tests passing
- formatting changes coming in next commit to isolate them
- removed Infinispan dependencies (not compile/test failures in any subproject as a result)
Andrea Boriero
The Geeky Asian
Sanne Grinovero
Guillaume Smet
Steve Ebersole
Scott Marlow