Store payloadvalueconverter to ODataOutputContext for performance ODa…

…ta#456
chinadragon0515 · Feb 1, 2016 · f2c7cd9 · f2c7cd9
1 parent e1f4ff0
commit f2c7cd9
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -47,4 +47,7 @@ sln/packages/
 
 # Test artifacts
 test/FunctionalTests/Tests/Data/Northwind/Northwind.csdl1.csdl
-test/FunctionalTests/Tests/Data/Northwind/Northwind.csdl2.csdl
+test/FunctionalTests/Tests/Data/Northwind/Northwind.csdl2.csdl
+
+# Symbol
+*.pdb
diff --git a/src/Microsoft.OData.Core/JsonLight/ODataJsonLightValueSerializer.cs b/src/Microsoft.OData.Core/JsonLight/ODataJsonLightValueSerializer.cs
@@ -350,7 +350,7 @@ public void WritePrimitiveValue(
             value = this.Model.ConvertToUnderlyingTypeIfUIntValue(value, expectedTypeReference);
 
             IEdmPrimitiveTypeReference actualTypeReference = EdmLibraryExtensions.GetPrimitiveTypeReference(value.GetType());
-            ODataPayloadValueConverter converter = this.Model.GetPayloadValueConverter();
+            ODataPayloadValueConverter converter = this.JsonLightOutputContext.PayloadValueConverter;
 
             // Skip validation if user has set custom PayloadValueConverter
             if (expectedTypeReference != null && converter.GetType() == typeof(ODataPayloadValueConverter))

diff --git a/src/Microsoft.OData.Core/ODataOutputContext.cs b/src/Microsoft.OData.Core/ODataOutputContext.cs
@@ -45,6 +45,9 @@ public abstract class ODataOutputContext : IDisposable
         /// <summary>The type resolver to use.</summary>
         private readonly EdmTypeResolver edmTypeResolver;
 
+        /// <summary>The payload value converter to use.</summary>
+        private readonly ODataPayloadValueConverter payloadValueConverter;
+
         /// <summary>
         /// Constructor.
         /// </summary>
@@ -72,6 +75,7 @@ protected ODataOutputContext(
             this.model = model ?? EdmCoreModel.Instance;
             this.urlResolver = urlResolver;
             this.edmTypeResolver = EdmTypeWriterResolver.Instance;
+            this.payloadValueConverter = this.model.GetPayloadValueConverter();
         }
 
         /// <summary>
@@ -141,6 +145,17 @@ internal EdmTypeResolver EdmTypeResolver
             }
         }
 
+        /// <summary>
+        /// The payload value converter to use
+        /// </summary>
+        internal ODataPayloadValueConverter PayloadValueConverter
+        {
+            get
+            {
+                return this.payloadValueConverter;
+            }
+        }
+
         /// <summary>
         /// The Context Url level used when writing.
         /// </summary>