Skip to content

Commit 1d8aed2

Browse files
committed
Do not try to acquire lock twice
1 parent 0585b68 commit 1d8aed2

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

xarray/backends/file_manager.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -240,11 +240,7 @@ def close(self, needs_lock: bool = True) -> None:
240240
file = self._cache.pop(self._key, default)
241241
if file is None:
242242
return
243-
if needs_lock and self._lock:
244-
with self._lock:
245-
file.close()
246-
else:
247-
file.close()
243+
file.close()
248244

249245
def __del__(self) -> None:
250246
# If we're the only CachingFileManger referencing a unclosed file,

0 commit comments

Comments
 (0)