Add local memory cache to Redis implementation to reduce roundtrips (#1214)

* Add local memory cache to Redis implementation to reduce roundtrips

* It seems better to keep GxRedis.cs unified for both .NET Framework and .NET Core.

* Applied Fazzato’s review suggestions

* Fix build error.

* Read ENABLE_MEMORY_CACHE from provider settings and apply it only in the default constructor used by CacheAPI.

* Refactor local cache settings in Redis implementation to support configurable maximum TTL and adjust local cache factor to 20%.

---------

Co-authored-by: claudiamurialdo <[email protected]>
(cherry picked from commit 6126f20)

# Conflicts:
#	dotnet/src/dotnetframework/Providers/Cache/GxRedis/GxRedis.cs

Author: claudiamurialdo

dotnet/src/dotnetframework/Providers/Cache/GxRedis/GxRedis.cs

@@ -7,9 +7,12 @@
 using GxClasses.Helpers;
 using Microsoft.Extensions.Caching.Memory;
 #endif
+using GeneXus.Encryption;
 using GeneXus.Services;
 using GeneXus.Utils;
 using StackExchange.Redis;
+using StackExchange.Redis.KeyspaceIsolation;
+using GeneXus.Configuration;
 
 namespace GeneXus.Cache
 {
@@ -21,9 +24,11 @@ public sealed class Redis : ICacheService2
 		IDatabase _redisDatabase;
 #if NETCORE
 		MemoryCache _localCache;
-		private const double DEFAULT_LOCAL_CACHE_FACTOR = 0.8;
-		private static readonly TimeSpan LOCAL_CACHE_PERSISTENT_KEY_TTL = TimeSpan.FromMinutes(5);
-	
+		private const double DEFAULT_LOCAL_CACHE_FACTOR = 0.2;
+		private TimeSpan MAX_LOCAL_CACHE_TTL;
+		private long MAX_LOCAL_CACHE_TTL_TICKS;
+		private const int MAX_LOCAL_CACHE_TTL_DEFAULT_MIMUTES = 5;
+
 #endif
 		ConfigurationOptions _redisConnectionOptions;
 		private const int REDIS_DEFAULT_PORT = 6379;
@@ -76,6 +81,18 @@ private void InitLocalCache(GXService providerService)
 			{
 				GXLogging.Debug(log, "Using Redis Hybrid mode with local memory cache.");
 				_localCache = new MemoryCache(new MemoryCacheOptions());
+				if (Config.GetValueOrEnvironmentVarOf("MAX_LOCAL_CACHE_TTL", out string maxCacheTtlMinutesStr) && long.TryParse(maxCacheTtlMinutesStr, out long maxCacheTtlMinutes))
+				{
+					MAX_LOCAL_CACHE_TTL = TimeSpan.FromMinutes(maxCacheTtlMinutes);
+					GXLogging.Debug(log, $"MAX_LOCAL_CACHE_TTL read from config: {MAX_LOCAL_CACHE_TTL}");
+				}
+				else
+				{
+					MAX_LOCAL_CACHE_TTL = TimeSpan.FromMinutes(MAX_LOCAL_CACHE_TTL_DEFAULT_MIMUTES);
+					GXLogging.Debug(log, $"MAX_LOCAL_CACHE_TTL using default value: {MAX_LOCAL_CACHE_TTL}");
+				}
+
+				MAX_LOCAL_CACHE_TTL_TICKS = MAX_LOCAL_CACHE_TTL.Ticks;
 			}
 			else
 			{
@@ -377,7 +394,13 @@ private TimeSpan LocalCacheTTL(int durationMinutes)
 		}
 		private TimeSpan LocalCacheTTL(TimeSpan? ttl)
 		{
-			return ttl.HasValue ? TimeSpan.FromTicks((long)(ttl.Value.Ticks * DEFAULT_LOCAL_CACHE_FACTOR)) : LOCAL_CACHE_PERSISTENT_KEY_TTL;
+			if (ttl.HasValue)
+			{
+				double ttlTicks = ttl.Value.Ticks * DEFAULT_LOCAL_CACHE_FACTOR;
+				if (ttlTicks < MAX_LOCAL_CACHE_TTL_TICKS)
+					return ttl.Value;
+			}
+			return MAX_LOCAL_CACHE_TTL;
 		}
 #endif
 		private void ClearKeyLocal(string key)
@@ -421,7 +444,7 @@ private void SetLocal<T>(string key, T value)
 		private void SetPersistentLocal(string cacheid, long? prefix)
 		{
 #if NETCORE
-			_localCache?.Set(cacheid, prefix, LocalCacheTTL(LOCAL_CACHE_PERSISTENT_KEY_TTL));
+			_localCache?.Set(cacheid, prefix, LocalCacheTTL(MAX_LOCAL_CACHE_TTL));
 #endif
 		}
 		private void SetLocal<T>(string key, T value, int duration)