Caching cause leaking of entities data from one session factory to another

I know this is a closed issue, but I thought I would answer my own question in case anybody finds this. For NH 2.0, ICacheProvider.BuildCache() still has the following signature:

ICache BuildCache(string regionName, IDictionary<string,string> properties)

However, the regionName that NH passes to the provider already includes the prefix. For example, if regionPrefix was "TestPfx", and NH wanted to create a region named "Default", it would read the regionPrefix and create a cache named "TestPfx.Default".

Also, I noticed that ICacheProvider has a Start() method with the following signature:

void Start(IDictionary<string, string> properties)

From here, you can use "Environment.CacheRegionPrefix" as the key to obtain the configured regionPrefix. Once you have the configured regionPrefix, follow the same naming pattern to configure your own caches.

Kind of neat.

I am in the process of implementing a cache provider and noticed that the ICacheProvider.BuildCache() still has the following signature for NH 2.0:

ICache BuildCache(string regionName, IDictionary<string,string> properties)

Is this the case? If so, how would I go about getting these changes applied to the 2.0 branch?

Many thanks?

Fixed in 2524

more specifically, I am talking about it not being used for entities.
For query cache, it is used, but not when storing the entity data itself.

Looks like this is really an issue, the caches are not using regionPrefix, although it is passed to them.