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
WINDUPRULE-381: hibernate 5.1-5.3 migration, RegionFactory class usag… #364
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added some notes on the <where>
parameter to better reflect this issue.
In case i missed something, please report it to me 😉
<where param=""> | ||
<matches pattern="" /> | ||
<where param="className"> | ||
<matches pattern="(cache.RegionFactory|cache.spi.RegionFactory|cache.spi.AbstractRegionFactory|testing.cache.CachingRegionFactory|cache.spi.support.RegionFactoryTemplate)" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i feel like we should cover "old" classes (CachingRegionFactory, EhCacheRegionFactory, InfinispanRegionFactory, JndiInfinispanRegionFactory, NoCachingRegionFactory, SingletonEhCacheRegionFactory
) that are not there anymore analysing org.hibernate.cache.spi.RegionFactory Hibernate 5.1 Javadoc because, afaiu, they removed some of these classes.
Co-Authored-By: m-brophy <36955467+m-brophy@users.noreply.github.com>
Co-Authored-By: m-brophy <36955467+m-brophy@users.noreply.github.com>
Co-Authored-By: m-brophy <36955467+m-brophy@users.noreply.github.com>
…ulesets into WINDUPRULE-381
followed up by #366 |
…e review
https://issues.jboss.org/browse/WINDUPRULE-381