[Storage] [Named Keywords] Blob Package (#41726)

Author: weirongw23-msft

sdk/storage/azure-storage-blob/azure/storage/blob/__init__.py

@@ -125,8 +125,8 @@ def upload_blob_to_url(
     :return: Blob-updated property dict (Etag and last modified)
     :rtype: dict(str, Any)
     """
-    with BlobClient.from_blob_url(blob_url, credential=credential) as client:
-        return cast(BlobClient, client).upload_blob(data=data, blob_type=BlobType.BLOCKBLOB, **kwargs)
+    with BlobClient.from_blob_url(blob_url, credential=credential) as client:  # pylint: disable=not-context-manager
+        return client.upload_blob(data=data, blob_type=BlobType.BLOCKBLOB, **kwargs)
 
 
 def _download_to_stream(client: BlobClient, handle: IO[bytes], **kwargs: Any) -> None:
@@ -194,7 +194,7 @@ def download_blob_from_url(
     :rtype: None
     """
     overwrite = kwargs.pop('overwrite', False)
-    with BlobClient.from_blob_url(blob_url, credential=credential) as client:
+    with BlobClient.from_blob_url(blob_url, credential=credential) as client: # pylint: disable=not-context-manager
         if hasattr(output, 'write'):
             _download_to_stream(client, cast(IO[bytes], output), **kwargs)
         else:

sdk/storage/azure-storage-blob/azure/storage/blob/_blob_client.py

@@ -9,7 +9,7 @@
 from datetime import datetime
 from functools import partial
 from typing import (
-    Any, AnyStr, cast, Dict, IO, Iterable, List, Optional, overload, Tuple, Union,
+    Any, AnyStr, cast, Dict, IO, Iterable, List, Optional, Tuple, Union,
     TYPE_CHECKING
 )
 from typing_extensions import Self
@@ -633,26 +633,6 @@ def upload_blob(
             return upload_page_blob(**options)
         return upload_append_blob(**options)
 
-    @overload
-    def download_blob(
-        self, offset: Optional[int] = None,
-        length: Optional[int] = None,
-        *,
-        encoding: str,
-        **kwargs: Any
-    ) -> StorageStreamDownloader[str]:
-        ...
-
-    @overload
-    def download_blob(
-        self, offset: Optional[int] = None,
-        length: Optional[int] = None,
-        *,
-        encoding: None = None,
-        **kwargs: Any
-    ) -> StorageStreamDownloader[bytes]:
-        ...
-
     @distributed_trace
     def download_blob(
         self, offset: Optional[int] = None,