Mmuzaf commented on code in PR #10157:
URL: https://github.com/apache/ignite/pull/10157#discussion_r942738985
##########
modules/clients/src/test/java/org/apache/ignite/internal/client/integration/ClientAbstractMultiNodeSelfTest.java:
##########
@@ -198,18 +196,13 @@ private CacheConfiguration cacheConfiguration(@NotNull
String cacheName) throws
cfg.setAtomicityMode(TRANSACTIONAL);
- switch (cacheName) {
- case DEFAULT_CACHE_NAME:
- cfg.setCacheMode(LOCAL);
- break;
- case PARTITIONED_CACHE_NAME:
- cfg.setCacheMode(PARTITIONED);
-
- cfg.setBackups(0);
- break;
- default:
- cfg.setCacheMode(REPLICATED);
- break;
+ if (PARTITIONED_CACHE_NAME.equals(cacheName)) {
Review Comment:
You're right. I've fixed this case and all the similar cases through the
source code.
##########
modules/platforms/dotnet/Apache.Ignite.Core/Impl/IgniteUtils.cs:
##########
@@ -231,5 +232,43 @@ public static int EncodePeekModes(CachePeekMode[] modes,
out bool hasPlatformCac
return res & ~platformCache;
}
+
+ /// <summary>
+ /// Converts integer code representation to CacheMode.
+ /// </summary>
+ /// <param name="code">Integer representation.</param>
+ /// <returns>Cache mode.</returns>
+ /// <exception cref="NotSupportedException">If fails.</exception>
+ public static CacheMode CacheModeFromInt(int code)
Review Comment:
Fixed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
