fix(model-server): IImmutableStore is supposed to return null values

Author: slisson

model-server/src/main/kotlin/org/modelix/model/server/store/IImmutableStore.kt

@@ -7,7 +7,7 @@ import org.modelix.model.IGenericKeyListener
  * Doesn't require transactions.
  */
 interface IImmutableStore<KeyT> {
-    fun getAll(keys: Set<KeyT>): Map<KeyT, String>
+    fun getAll(keys: Set<KeyT>): Map<KeyT, String?>
     fun addAll(entries: Map<KeyT, String>)
     fun getIfCached(key: KeyT): String?
 }

model-server/src/main/kotlin/org/modelix/model/server/store/IgniteStoreClient.kt

@@ -247,13 +247,9 @@ class IgniteStoreClient(jdbcProperties: Properties? = null, private val inmemory
 
     override fun getImmutableStore(): IImmutableStore<ObjectInRepository> {
         return object : IImmutableStore<ObjectInRepository> {
-            override fun getAll(keys: Set<ObjectInRepository>): Map<ObjectInRepository, String> {
+            override fun getAll(keys: Set<ObjectInRepository>): Map<ObjectInRepository, String?> {
                 keys.forEach { require(HashUtil.isSha256(it.key)) { "Not an immutable object: $it" } }
-                return cache.getAll(keys).mapValues {
-                    val value = it.value
-                    if (value == null) throw ObjectValueNotFoundException(it.key.key)
-                    value
-                }
+                return cache.getAll(keys)
             }
 
             override fun addAll(entries: Map<ObjectInRepository, String>) {

model-server/src/main/kotlin/org/modelix/model/server/store/InMemoryStoreClient.kt

@@ -2,7 +2,6 @@ package org.modelix.model.server.store
 
 import org.modelix.model.IGenericKeyListener
 import org.modelix.model.persistent.HashUtil
-import org.modelix.model.server.handlers.ObjectValueNotFoundException
 import org.slf4j.LoggerFactory
 
 fun generateId(idStr: String?): Long {
@@ -138,14 +137,10 @@ class InMemoryStoreClient : IsolatingStore, ITransactionManager, IRepositoryAwar
 
     override fun getImmutableStore(): IImmutableStore<ObjectInRepository> {
         return object : IImmutableStore<ObjectInRepository> {
-            override fun getAll(keys: Set<ObjectInRepository>): Map<ObjectInRepository, String> {
+            override fun getAll(keys: Set<ObjectInRepository>): Map<ObjectInRepository, String?> {
                 keys.forEach { require(HashUtil.isSha256(it.key)) { "Not an immutable object: $it" } }
                 @OptIn(RequiresTransaction::class)
-                return runRead { this@InMemoryStoreClient.getAll(keys) }.mapValues {
-                    val value = it.value
-                    if (value == null) throw ObjectValueNotFoundException(it.key.key)
-                    value
-                }
+                return runRead { this@InMemoryStoreClient.getAll(keys) }
             }
 
             override fun addAll(entries: Map<ObjectInRepository, String>) {

model-server/src/main/kotlin/org/modelix/model/server/store/StoreClientAdapter.kt

@@ -90,7 +90,7 @@ abstract class StoreClientAdapter(val client: IsolatingStore) : IStoreClient {
     override fun getImmutableStore(): IImmutableStore<String> {
         val immutableStore = client.getImmutableStore()
         return object : IImmutableStore<String> {
-            override fun getAll(keys: Set<String>): Map<String, String> {
+            override fun getAll(keys: Set<String>): Map<String, String?> {
                 val fromRepository = immutableStore.getAll(keys.map { it.withRepoScope() }.toSet()).mapKeys { it.key.key }
                 if (getRepositoryId() == null) return fromRepository