Second level cache (still?) not working for inherited domains

[dbaylerg]

I am still having an issue with the Second Level Cache not working for inherited domains.

I have put together a sample project which demonstrates the issue as I see it, or perhaps I have configured something incorrectly.

https://github.com/dbaylerg/GormCacheIssue

The project was created with Grails 4.0.13 with updated dependencies to include the latest fixes from grails/grails-data-hibernate5#483

What I am seeing is that my inherited domains still are not cached. Any help or suggestions would be appreciated.

[dbaylerg]

I believe I included the fix for https://github.com/grails/gorm-hibernate5/issues/655 but they may still be related. The difference is that in my example I do not use an abstract parent class.

[purpleraven]

@dbaylerg as I can see my fix not in released version

[dbaylerg]

@purpleraven I looked at the commit & release history for the 7.0.x branch and it looks to me that it was included in org.grails.plugins:hibernate5 7.0.7 release.

7.0.6 was released on Jan 12, 2022, your fix was committed on Jan 19th, 2022, and 7.0.7 was released on Dec 20, 2022.

Am I looking at the wrong branch?

[purpleraven]

@dbaylerg the fix presents in the 7.0.x branch, but not in 7.1.x and higher.

@puneetbehl Should I create a PRs for another branches?

https://github.com/grails/gorm-hibernate5/pull/483/files

[dbaylerg]

@purpleraven @puneetbehl

First of all, thank you, I appreciate the responses but somehow we seem to be talking about different versions.

My test application is built and tested with the dependency
compile "org.grails.plugins:hibernate5:7.0.7"

So, I believe I am testing with the 7.0.x branch not the 7.1.x branch. The fix for bug #483 appears to included in the 7.0.7 version but does not seem to fix the issue.

If I am mistaken about this I would appreciate if you can point out what I am missing or got wrong.

[purpleraven]

I have fixed just one of case.
For make it work you have to add mapping cache property to base class.

[dbaylerg]

@purpleraven All of the classes, both base class and sub-classes, have

static mapping = {
    cache true
}

[purpleraven]

Could you prepare a test application, please?

[dbaylerg]

@purpleraven The test application is linked above but here it is https://github.com/dbaylerg/GormCacheIssue

[dbaylerg]

@purpleraven Any news on this issue?

[purpleraven]

@dbaylerg no, vacation time...

[dbaylerg]

@purpleraven It has been awhile but I thought I would check in one last time before abandoning hope for an official fix.

[purpleraven]

@dbaylerg I really forgot about the case. Sorry.