I do still have one concern with changing the calls to the cache when the Session[1] is not involved in a transaction (CacheTransactionContext is null). For impls that need access to Session there they would have no way to get the Session from the null CacheTransactionContext reference. Perhaps it is better to continue to pass the Session and expose the current SharedSessionContractImplementor from there.

Andrea Boriero You have an opinion?

[1] SharedSessionContractImplementor