Apply shared cache options in the RemoveNode method

Author: alex-kulakov

Orm/Xtensive.Orm/Orm/StorageNodeManager.cs

@@ -52,14 +52,22 @@ public async Task<bool> AddNodeAsync([NotNull] NodeConfiguration configuration,
     /// </summary>
     /// <param name="nodeId">Node identifier.</param>
     /// <param name="clearQueryCache">
-    /// if <see langword="true"/> then cached queries dedicated to the removing node will be removed from cache as well. By default <see langword="false"/>.
+    /// If <see langword="true"/> then cached queries dedicated to the removing node will be removed from cache as well. By default <see langword="false"/>.
+    /// Also note, that this parameter has no effect in case when both <see cref="DomainConfiguration.ShareStorageSchemaOverNodes"/>
+    /// and <see cref="DomainConfiguration.PreferTypeIdsAsQueryParameters"/> options are turned on because cache  will contain one
+    /// item for all storage node instead of copies of same query per each storage node.
     /// </param>
     /// <returns>True if node was removed, otherwise false.</returns>
     public bool RemoveNode([NotNull] string nodeId, bool clearQueryCache = false)
     {
       var removeResult = handlers.StorageNodeRegistry.Remove(nodeId);
 
       if (removeResult && clearQueryCache) {
+        var domainConfig = handlers.Domain.Configuration;
+        if (domainConfig.ShareStorageSchemaOverNodes && domainConfig.PreferTypeIdsAsQueryParameters) {
+          return removeResult;
+        }
+
         var queryCache = (Caching.FastConcurrentLruCache<object, Pair<object, Linq.ParameterizedQuery>>) handlers.Domain.QueryCache;
         foreach (var key in queryCache.Keys.Where(k => k is Pair<object, string> pair && pair.Second == nodeId).ToChainedBuffer()) {
           queryCache.RemoveKey(key);