Skip to content

Commit fae4917

Browse files
committed
Renamed IBatchableReadWriteCache to IBatchableCache and IBatchableReadCache to IBatchableReadOnlyCache
1 parent 1f6db2a commit fae4917

17 files changed

+66
-66
lines changed

src/AsyncGenerator.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -258,8 +258,8 @@ methodRules:
258258
name: Lock
259259
- containingType: NHibernate.Cache.ICache
260260
name: Unlock
261-
- containingType: NHibernate.Cache.IBatchableReadCache
262-
- containingType: NHibernate.Cache.IBatchableReadWriteCache
261+
- containingType: NHibernate.Cache.IBatchableReadOnlyCache
262+
- containingType: NHibernate.Cache.IBatchableCache
263263
name: Cache
264264
- filters:
265265
- containingNamespace: NHibernate

src/NHibernate.Test/Async/CacheTest/Caches/BatchableCache.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919

2020
namespace NHibernate.Test.CacheTest.Caches
2121
{
22-
public partial class BatchableCache : ICache, IBatchableReadWriteCache
22+
public partial class BatchableCache : ICache, IBatchableCache
2323
{
2424

2525
public Task PutManyAsync(object[] keys, object[] values, CancellationToken cancellationToken)

src/NHibernate.Test/CacheTest/Caches/BatchableCache.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
namespace NHibernate.Test.CacheTest.Caches
1111
{
12-
public partial class BatchableCache : ICache, IBatchableReadWriteCache
12+
public partial class BatchableCache : ICache, IBatchableCache
1313
{
1414
private readonly IDictionary _hashtable = new Hashtable();
1515

src/NHibernate/Async/Cache/IBatchableWriteCache.cs renamed to src/NHibernate/Async/Cache/IBatchableCache.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ namespace NHibernate.Cache
1616
{
1717
using System.Threading.Tasks;
1818
using System.Threading;
19-
public partial interface IBatchableReadWriteCache : IBatchableReadCache
19+
public partial interface IBatchableCache : IBatchableReadOnlyCache
2020
{
2121
/// <summary>
2222
/// Add multiple objects to the cache.

src/NHibernate/Async/Cache/IBatchableReadCache.cs renamed to src/NHibernate/Async/Cache/IBatchableReadOnlyCache.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ namespace NHibernate.Cache
1616
{
1717
using System.Threading.Tasks;
1818
using System.Threading;
19-
public partial interface IBatchableReadCache
19+
public partial interface IBatchableReadOnlyCache
2020
{
2121
/// <summary>
2222
/// Get multiple objects from the cache.

src/NHibernate/Async/Cache/NonstrictReadWriteCache.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public async Task<object> GetAsync(CacheKey key, long txTimestamp, CancellationT
4646

4747
public Task<object[]> GetManyAsync(CacheKey[] keys, long timestamp, CancellationToken cancellationToken)
4848
{
49-
if (_batchableReadCache == null)
49+
if (_batchableReadOnlyCache == null)
5050
{
5151
throw new InvalidOperationException($"Cache {cache.GetType()} does not support batching get operation");
5252
}
@@ -61,7 +61,7 @@ async Task<object[]> InternalGetManyAsync()
6161
{
6262
log.Debug("Cache lookup: {0}", string.Join(",", keys.AsEnumerable()));
6363
}
64-
var results = await (_batchableReadCache.GetManyAsync(keys.Select(o => (object) o).ToArray(), cancellationToken)).ConfigureAwait(false);
64+
var results = await (_batchableReadOnlyCache.GetManyAsync(keys.Select(o => (object) o).ToArray(), cancellationToken)).ConfigureAwait(false);
6565
if (!log.IsDebugEnabled())
6666
{
6767
return results;
@@ -80,7 +80,7 @@ async Task<object[]> InternalGetManyAsync()
8080
public Task<bool[]> PutManyAsync(CacheKey[] keys, object[] values, long timestamp, object[] versions, IComparer[] versionComparers,
8181
bool[] minimalPuts, CancellationToken cancellationToken)
8282
{
83-
if (_batchableReadWriteCache == null)
83+
if (_batchableCache == null)
8484
{
8585
throw new InvalidOperationException($"Cache {cache.GetType()} does not support batching operations");
8686
}
@@ -111,7 +111,7 @@ async Task<bool[]> InternalPutManyAsync()
111111
var skipKeyIndexes = new HashSet<int>();
112112
if (checkKeys.Any())
113113
{
114-
var objects = await (_batchableReadWriteCache.GetManyAsync(checkKeys.ToArray(), cancellationToken)).ConfigureAwait(false);
114+
var objects = await (_batchableCache.GetManyAsync(checkKeys.ToArray(), cancellationToken)).ConfigureAwait(false);
115115
for (var i = 0; i < objects.Length; i++)
116116
{
117117
if (objects[i] != null)
@@ -143,7 +143,7 @@ async Task<bool[]> InternalPutManyAsync()
143143
putValues[j++] = values[i];
144144
result[i] = true;
145145
}
146-
await (_batchableReadWriteCache.PutManyAsync(putKeys, putValues, cancellationToken)).ConfigureAwait(false);
146+
await (_batchableCache.PutManyAsync(putKeys, putValues, cancellationToken)).ConfigureAwait(false);
147147
return result;
148148
}
149149
}

src/NHibernate/Async/Cache/ReadOnlyCache.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public async Task<object> GetAsync(CacheKey key, long timestamp, CancellationTok
3434

3535
public Task<object[]> GetManyAsync(CacheKey[] keys, long timestamp, CancellationToken cancellationToken)
3636
{
37-
if (_batchableReadCache == null)
37+
if (_batchableReadOnlyCache == null)
3838
{
3939
throw new InvalidOperationException($"Cache {cache.GetType()} does not support batching get operation");
4040
}
@@ -49,7 +49,7 @@ async Task<object[]> InternalGetManyAsync()
4949
{
5050
log.Debug("Cache lookup: {0}", string.Join(",", keys.AsEnumerable()));
5151
}
52-
var results = await (_batchableReadCache.GetManyAsync(keys.Select(o => (object) o).ToArray(), cancellationToken)).ConfigureAwait(false);
52+
var results = await (_batchableReadOnlyCache.GetManyAsync(keys.Select(o => (object) o).ToArray(), cancellationToken)).ConfigureAwait(false);
5353
if (!log.IsDebugEnabled())
5454
{
5555
return results;
@@ -84,7 +84,7 @@ public Task<ISoftLock> LockAsync(CacheKey key, object version, CancellationToken
8484
public Task<bool[]> PutManyAsync(CacheKey[] keys, object[] values, long timestamp, object[] versions, IComparer[] versionComparers,
8585
bool[] minimalPuts, CancellationToken cancellationToken)
8686
{
87-
if (_batchableReadWriteCache == null)
87+
if (_batchableCache == null)
8888
{
8989
throw new InvalidOperationException($"Cache {cache.GetType()} does not support batching operations");
9090
}
@@ -115,7 +115,7 @@ async Task<bool[]> InternalPutManyAsync()
115115
var skipKeyIndexes = new HashSet<int>();
116116
if (checkKeys.Any())
117117
{
118-
var objects = await (_batchableReadWriteCache.GetManyAsync(checkKeys.Select(o => (object) o).ToArray(), cancellationToken)).ConfigureAwait(false);
118+
var objects = await (_batchableCache.GetManyAsync(checkKeys.Select(o => (object) o).ToArray(), cancellationToken)).ConfigureAwait(false);
119119
for (var i = 0; i < objects.Length; i++)
120120
{
121121
if (objects[i] != null)
@@ -147,7 +147,7 @@ async Task<bool[]> InternalPutManyAsync()
147147
putValues[j++] = values[i];
148148
result[i] = true;
149149
}
150-
await (_batchableReadWriteCache.PutManyAsync(putKeys, putValues, cancellationToken)).ConfigureAwait(false);
150+
await (_batchableCache.PutManyAsync(putKeys, putValues, cancellationToken)).ConfigureAwait(false);
151151
return result;
152152
}
153153
}

src/NHibernate/Async/Cache/ReadWriteCache.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ public async Task<object> GetAsync(CacheKey key, long txTimestamp, CancellationT
9292

9393
public Task<object[]> GetManyAsync(CacheKey[] keys, long timestamp, CancellationToken cancellationToken)
9494
{
95-
if (_batchableReadCache == null)
95+
if (_batchableReadOnlyCache == null)
9696
{
9797
throw new InvalidOperationException($"Cache {cache.GetType()} does not support batching get operation");
9898
}
@@ -110,7 +110,7 @@ async Task<object[]> InternalGetManyAsync()
110110
var result = new object[keys.Length];
111111
using (await _lockObjectAsync.LockAsync())
112112
{
113-
var lockables = await (_batchableReadCache.GetManyAsync(keys.Select(o => (object) o).ToArray(), cancellationToken)).ConfigureAwait(false);
113+
var lockables = await (_batchableReadOnlyCache.GetManyAsync(keys.Select(o => (object) o).ToArray(), cancellationToken)).ConfigureAwait(false);
114114
for (var i = 0; i < lockables.Length; i++)
115115
{
116116
var lockable = (ILockable) lockables[i];
@@ -183,7 +183,7 @@ public async Task<ISoftLock> LockAsync(CacheKey key, object version, Cancellatio
183183
public Task<bool[]> PutManyAsync(CacheKey[] keys, object[] values, long timestamp, object[] versions, IComparer[] versionComparers,
184184
bool[] minimalPuts, CancellationToken cancellationToken)
185185
{
186-
if (_batchableReadWriteCache == null)
186+
if (_batchableCache == null)
187187
{
188188
throw new InvalidOperationException($"Cache {cache.GetType()} does not support batching operations");
189189
}
@@ -211,9 +211,9 @@ async Task<bool[]> InternalPutManyAsync()
211211
var keysArr = keys.Cast<object>().ToArray();
212212
try
213213
{
214-
await (_batchableReadWriteCache.LockManyAsync(keysArr, cancellationToken)).ConfigureAwait(false);
214+
await (_batchableCache.LockManyAsync(keysArr, cancellationToken)).ConfigureAwait(false);
215215
var putBatch = new Dictionary<object, object>();
216-
var lockables = await (_batchableReadWriteCache.GetManyAsync(keysArr, cancellationToken)).ConfigureAwait(false);
216+
var lockables = await (_batchableCache.GetManyAsync(keysArr, cancellationToken)).ConfigureAwait(false);
217217
for (var i = 0; i < keys.Length; i++)
218218
{
219219
var key = keys[i];
@@ -249,12 +249,12 @@ async Task<bool[]> InternalPutManyAsync()
249249

250250
if (putBatch.Count > 0)
251251
{
252-
await (_batchableReadWriteCache.PutManyAsync(putBatch.Keys.ToArray(), putBatch.Values.ToArray(), cancellationToken)).ConfigureAwait(false);
252+
await (_batchableCache.PutManyAsync(putBatch.Keys.ToArray(), putBatch.Values.ToArray(), cancellationToken)).ConfigureAwait(false);
253253
}
254254
}
255255
finally
256256
{
257-
await (_batchableReadWriteCache.UnlockManyAsync(keysArr, cancellationToken)).ConfigureAwait(false);
257+
await (_batchableCache.UnlockManyAsync(keysArr, cancellationToken)).ConfigureAwait(false);
258258
}
259259
}
260260
return result;

src/NHibernate/Async/Engine/BatchFetchQueue.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ internal async Task<object[]> GetCollectionBatchAsync(ICollectionPersister colle
6666
// List of collection entries that haven't been checked for their existance in the cache. Besides the collection entry,
6767
// the index where the entry was found is also stored in order to correctly order the returning keys.
6868
var collectionKeys = new List<KeyValuePair<KeyValuePair<CollectionEntry, IPersistentCollection>, int>>(batchSize);
69-
var batchableCache = collectionPersister.Cache?.Cache as IBatchableReadCache;
69+
var batchableCache = collectionPersister.Cache?.Cache as IBatchableReadOnlyCache;
7070

7171
if (!batchLoadableCollections.TryGetValue(collectionPersister.Role, out var map))
7272
{
@@ -258,7 +258,7 @@ internal async Task<object[]> GetEntityBatchAsync(IEntityPersister persister, ob
258258
// List of entity keys that haven't been checked for their existance in the cache. Besides the entity key,
259259
// the index where the key was found is also stored in order to correctly order the returning keys.
260260
var entityKeys = new List<KeyValuePair<EntityKey, int>>(batchSize);
261-
var batchableCache = persister.Cache?.Cache as IBatchableReadCache;
261+
var batchableCache = persister.Cache?.Cache as IBatchableReadOnlyCache;
262262

263263
if (!batchLoadableEntityKeys.TryGetValue(persister.EntityName, out var set))
264264
{
@@ -412,7 +412,7 @@ private async Task<bool> IsCachedAsync(object collectionKey, ICollectionPersiste
412412
/// <param name="cancellationToken">A cancellation token that can be used to cancel the work</param>
413413
/// <returns>An array of booleans that contains the result for each key.</returns>
414414
private async Task<bool[]> AreCachedAsync(List<KeyValuePair<EntityKey, int>> entityKeys, int[] keyIndexes, IEntityPersister persister,
415-
IBatchableReadCache batchableCache, bool checkCache, CancellationToken cancellationToken)
415+
IBatchableReadOnlyCache batchableCache, bool checkCache, CancellationToken cancellationToken)
416416
{
417417
cancellationToken.ThrowIfCancellationRequested();
418418
var result = new bool[keyIndexes.Length];
@@ -450,7 +450,7 @@ private async Task<bool[]> AreCachedAsync(List<KeyValuePair<EntityKey, int>> ent
450450
/// <param name="cancellationToken">A cancellation token that can be used to cancel the work</param>
451451
/// <returns>An array of booleans that contains the result for each key.</returns>
452452
private async Task<bool[]> AreCachedAsync(List<KeyValuePair<KeyValuePair<CollectionEntry, IPersistentCollection>, int>> collectionKeys,
453-
int[] keyIndexes, ICollectionPersister persister, IBatchableReadCache batchableCache,
453+
int[] keyIndexes, ICollectionPersister persister, IBatchableReadOnlyCache batchableCache,
454454
bool checkCache, CancellationToken cancellationToken)
455455
{
456456
cancellationToken.ThrowIfCancellationRequested();

src/NHibernate/Cache/IBatchableWriteCache.cs renamed to src/NHibernate/Cache/IBatchableCache.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ namespace NHibernate.Cache
1515
/// All implementations <em>must</em> be threadsafe.
1616
/// </para>
1717
/// </remarks>
18-
public partial interface IBatchableReadWriteCache : IBatchableReadCache
18+
public partial interface IBatchableCache : IBatchableReadOnlyCache
1919
{
2020
/// <summary>
2121
/// Add multiple objects to the cache.

0 commit comments

Comments
 (0)