Remove lock kwarg (#5912)

Co-authored-by: Illviljan <[email protected]>

Author: TomNicholas

doc/whats-new.rst

@@ -31,6 +31,8 @@ Breaking changes
 
 Deprecations
 ~~~~~~~~~~~~
+- Removed the lock kwarg from the zarr and pydap backends, completing the deprecation cycle started in :issue:`5256`.
+  By `Tom Nicholas <https://github.com/TomNicholas>`_.
 
 
 Bug fixes

xarray/backends/pydap_.py

@@ -1,5 +1,3 @@
-import warnings
-
 import numpy as np
 
 from ..core import indexing
@@ -126,15 +124,7 @@ def open_dataset(
         use_cftime=None,
         decode_timedelta=None,
         session=None,
-        lock=None,
     ):
-        # TODO remove after v0.19
-        if lock is not None:
-            warnings.warn(
-                "The kwarg 'lock' has been deprecated for this backend, and is now "
-                "ignored. In the future passing lock will raise an error.",
-                DeprecationWarning,
-            )
 
         store = PydapDataStore.open(
             filename_or_obj,

xarray/backends/zarr.py

@@ -810,15 +810,7 @@ def open_dataset(
         chunk_store=None,
         storage_options=None,
         stacklevel=3,
-        lock=None,
     ):
-        # TODO remove after v0.19
-        if lock is not None:
-            warnings.warn(
-                "The kwarg 'lock' has been deprecated for this backend, and is now "
-                "ignored. In the future passing lock will raise an error.",
-                DeprecationWarning,
-            )
 
         filename_or_obj = _normalize_path(filename_or_obj)
         store = ZarrStore.open_group(