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

GPHIB-18: Upgrade ehcache dependency to 2.9.0 #6

Merged
merged 2 commits into from
Nov 20, 2014

Conversation

candrews
Copy link
Contributor

Upgrade the ehcache dependency to 2.8.1.

@graemerocher
Copy link
Member

Upgrading to 2.8.1 caused various test failures https://travis-ci.org/grails-plugins/grails-hibernate4-plugin/builds/27287652

It doesn't seem the cache factory implementation is compatible?

@candrews candrews changed the title GPHIB-18: Upgrade ehcache dependency to 2.8.1 GPHIB-18: Upgrade ehcache dependency to 2.8.3 Jul 27, 2014
@candrews
Copy link
Contributor Author

@graemerocher I've updated this pull request. All tests pass and it seems to work nicely.

The build is currently failing because the master build is failing due to this commit: 3a59907
If you revert that commit, all tests pass. I think this is ready for merging.

@candrews
Copy link
Contributor Author

Just rebased and re-ran all the tests to ensure this patch still works. It's ready to merge.

@candrews candrews changed the title GPHIB-18: Upgrade ehcache dependency to 2.8.3 GPHIB-18: Upgrade ehcache dependency to 2.8.4 Sep 23, 2014
@candrews
Copy link
Contributor Author

Just updated the pull request for ehcache 2.8.4

…egionFactory

The tests rely on there being exactly one unnamed cache. Earlier versions of
EhCache are more tolerant and will reuse an existing unnamed cache if an
attempt is made to create another unnamed cache (or an attempt is made to
create another named cache with the same name), but later versions will throw an
exception. By using SingletonEhCacheProvider, the same cache is always used,
which is what earlier versions of EhCache do anyways, and prevents problems with
later versions.
@candrews candrews changed the title GPHIB-18: Upgrade ehcache dependency to 2.8.4 GPHIB-18: Upgrade ehcache dependency to 2.9.0 Oct 24, 2014
@candrews
Copy link
Contributor Author

Just rebased - still ready for merging.

@osscontributor
Copy link
Member

Craig has asked that we either accept or reject this so until one of us can take a closer look and do some testing I will go ahead and reject it.

Thanks for your help.

osscontributor pushed a commit that referenced this pull request Nov 20, 2014
GPHIB-18: Upgrade ehcache dependency to 2.9.0
@osscontributor osscontributor merged commit b9aa3f7 into grails-plugins:master Nov 20, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants