Model serializer caching.

tomchristie · tomchristie · commit c06a82d0531f · 2014-12-19T16:02:27.000Z
diff --git a/rest_framework/serializers.py b/rest_framework/serializers.py
@@ -682,6 +682,10 @@ def raise_errors_on_nested_writes(method_name, serializer, validated_data):
     )
 
 
+MODEL_SERIALIZER_FIELDS_CACHE = {}
+MODEL_SERIALIZER_VALIDATORS_CACHE = {}
+
+
 class ModelSerializer(Serializer):
     """
     A `ModelSerializer` is just a regular `Serializer`, except that:
@@ -802,6 +806,11 @@ def get_fields(self):
         Return the dict of field names -> field instances that should be
         used for `self.fields` when instantiating the serializer.
         """
+        cls = self.__class__
+
+        if cls in MODEL_SERIALIZER_FIELDS_CACHE:
+            return copy.deepcopy(MODEL_SERIALIZER_FIELDS_CACHE[cls])
+
         declared_fields = copy.deepcopy(self._declared_fields)
         model = getattr(self.Meta, 'model')
         depth = getattr(self.Meta, 'depth', 0)
@@ -837,6 +846,8 @@ def get_fields(self):
         # Add in any hidden fields.
         ret.update(hidden_fields)
 
+        MODEL_SERIALIZER_FIELDS_CACHE[cls] = ret
+
         return ret
 
     # Methods for determining the set of field names to include...
@@ -1217,12 +1228,21 @@ def get_validators(self):
         if validators is not None:
             return validators[:]
 
+        cls = self.__class__
+
+        if cls in MODEL_SERIALIZER_VALIDATORS_CACHE:
+            return MODEL_SERIALIZER_VALIDATORS_CACHE[cls][:]
+
         # Otherwise use the default set of validators.
-        return (
+        validators = (
             self.get_unique_together_validators() +
             self.get_unique_for_date_validators()
         )
 
+        MODEL_SERIALIZER_VALIDATORS_CACHE[cls] = validators
+
+        return validators
+
     def get_unique_together_validators(self):
         """
         Determine a default set of validators for any unique_together contraints.
