Fixes copy issue (#34615)

Author: mrm9084

sdk/appconfiguration/azure-appconfiguration-provider/CHANGELOG.md

@@ -12,6 +12,8 @@
 
 ### Bugs Fixed
 
+* Fixes issue where loading configurations were slower do to returning a copy of the configurations.
+
 ### Other Changes
 
 ## 1.1.0 (2024-01-29)

sdk/appconfiguration/azure-appconfiguration-provider/azure/appconfiguration/provider/_azureappconfigurationprovider.py

@@ -3,7 +3,6 @@
 # Licensed under the MIT License. See License.txt in the project root for
 # license information.
 # -------------------------------------------------------------------------
-import copy
 import os
 import json
 import random
@@ -690,7 +689,7 @@ def keys(self) -> KeysView[str]:
         :rtype: KeysView[str]
         """
         with self._update_lock:
-            return copy.deepcopy(self._dict).keys()
+            return self._dict.keys()
 
     def items(self) -> ItemsView[str, Union[str, Mapping[str, Any]]]:
         """
@@ -701,7 +700,7 @@ def items(self) -> ItemsView[str, Union[str, Mapping[str, Any]]]:
         :rtype: ItemsView[str, Union[str, Mapping[str, Any]]]
         """
         with self._update_lock:
-            return copy.deepcopy(self._dict).items()
+            return self._dict.items()
 
     def values(self) -> ValuesView[Union[str, Mapping[str, Any]]]:
         """
@@ -713,7 +712,7 @@ def values(self) -> ValuesView[Union[str, Mapping[str, Any]]]:
         :rtype: ValuesView[Union[str, Mapping[str, Any]]]
         """
         with self._update_lock:
-            return copy.deepcopy((self._dict)).values()
+            return (self._dict).values()
 
     @overload
     def get(self, key: str, default: None = None) -> Union[str, JSON, None]:
@@ -734,7 +733,7 @@ def get(self, key: str, default: Optional[Union[str, JSON, _T]] = None) -> Union
         :rtype: Union[str, JSON]
         """
         with self._update_lock:
-            return copy.deepcopy(self._dict).get(key, default)
+            return self._dict.get(key, default)
 
     def __eq__(self, other: Any) -> bool:
         if not isinstance(other, AzureAppConfigurationProvider):

sdk/appconfiguration/azure-appconfiguration-provider/azure/appconfiguration/provider/aio/_azureappconfigurationproviderasync.py

@@ -4,7 +4,6 @@
 # license information.
 # -------------------------------------------------------------------------
 import json
-import copy
 from threading import Lock
 import datetime
 import logging
@@ -583,7 +582,7 @@ def keys(self) -> KeysView[str]:
         :return: A list of keys loaded from Azure App Configuration.
         :rtype: KeysView[str]
         """
-        return copy.deepcopy(self._dict).keys()
+        return self._dict.keys()
 
     def items(self) -> ItemsView[str, Union[str, Mapping[str, Any]]]:
         """
@@ -593,7 +592,7 @@ def items(self) -> ItemsView[str, Union[str, Mapping[str, Any]]]:
         :return: A set-like object of key-value pairs loaded from Azure App Configuration.
         :rtype: ItemsView[str, Union[str, Mapping[str, Any]]]
         """
-        return copy.deepcopy(self._dict).items()
+        return self._dict.items()
 
     def values(self) -> ValuesView[Union[str, Mapping[str, Any]]]:
         """
@@ -604,7 +603,7 @@ def values(self) -> ValuesView[Union[str, Mapping[str, Any]]]:
          based on there content type.
         :rtype: ValuesView[Union[str, Mapping[str, Any]]]
         """
-        return copy.deepcopy(self._dict).values()
+        return self._dict.values()
 
     @overload
     def get(self, key: str, default: None = None) -> Union[str, JSON, None]:
@@ -624,7 +623,7 @@ def get(self, key: str, default: Optional[Union[str, JSON, _T]] = None) -> Union
         :return: The value of the specified key.
         :rtype: Union[str, JSON]
         """
-        return copy.deepcopy(self._dict).get(key, default)
+        return self._dict.get(key, default)
 
     def __eq__(self, other: Any) -> bool:
         if not isinstance(other, AzureAppConfigurationProvider):