Alex Zaytsev

Another thought -- can `CheckCache` be incorporated into `CacheBase`?

> incorporating CheckCache in CacheBase, I do not get it right now, I will have another look. Now most of our calls look like this: ```csharp // prop definition simplified...

> I thought you wanted me to put something in the CacheBase class, but instead you refer to the CacheBase typed property versus the ICache one. There were 2 suggestions...

Can you explain why you need additional property `InternalCache`?

> I have added a commit to remove InternalCache property. I have not had anything against this property. I was just wondering why it is needed. The explanation and reasoning...

I lean on agreeing with @bahusoid on the issue. We probably should release the 5.3.4 without this change.

> Nhibernate 5.3.12 @FouadSsitou does it happen on other versions?

Why not just `ORDER BY 1` and fix tests?

Let's discuss everything here. It is hard to follow conversation when it is spread in multiple places. The behavior here explicitly falls into undefined behavior category according to documentation and...

@fabiomaulo do you have a stack trace?