Merge pull request #334 from DataObjects-NET/7.1-persistrequest-no-nc-usage

PersistRequest: No NodeConfiguration usage

Author: alex-kulakov

ChangeLog/7.1.1-dev.txt

@@ -1 +1,2 @@
-[main] Improved memory efficency of cached queries.
+[main] Improved memory efficency of cached queries.
+[main] One of PersistRequest's constructors marked obsolete since it is no loger in use

Orm/Xtensive.Orm/Orm/Providers/Requests/PersistRequest.cs

@@ -28,8 +28,6 @@ public sealed class PersistRequest
 
     public IEnumerable<PersistParameterBinding> ParameterBindings { get; private set; }
 
-    internal NodeConfiguration NodeConfiguration { get; private set; }
-
     public SqlCompilationResult GetCompiledStatement()
     {
       if (compiledStatement==null)
@@ -50,10 +48,21 @@ public void Prepare()
 
     public PersistRequest(
       StorageDriver driver, SqlStatement statement, IEnumerable<PersistParameterBinding> parameterBindings)
-      : this(driver, statement, parameterBindings, null)
     {
+      ArgumentValidator.EnsureArgumentNotNull(driver, "driver");
+      ArgumentValidator.EnsureArgumentNotNull(statement, "statement");
+
+      var compileUnit = statement as ISqlCompileUnit;
+      if (compileUnit == null)
+        throw new ArgumentException("Statement is not ISqlCompileUnit");
+
+      this.driver = driver;
+      Statement = statement;
+      CompileUnit = compileUnit;
+      ParameterBindings = ParameterBinding.NormalizeBindings(parameterBindings);
     }
 
+    [Obsolete]
     public PersistRequest(
       StorageDriver driver, SqlStatement statement, IEnumerable<PersistParameterBinding> parameterBindings, NodeConfiguration nodeConfiguration)
     {
@@ -68,7 +77,6 @@ public PersistRequest(
       Statement = statement;
       CompileUnit = compileUnit;
       ParameterBindings = ParameterBinding.NormalizeBindings(parameterBindings);
-      NodeConfiguration = nodeConfiguration;
     }
   }
 }