Skip to content

Commit

Permalink
Remove lru_cache on method
Browse files Browse the repository at this point in the history
lru_cache on methods doesn't work properly. Furthermore if we
need to cache the collection type we should be caching it
elsewhere inside registry.
  • Loading branch information
timj committed Jul 21, 2023
1 parent 2aec315 commit 66b190f
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions python/lsst/daf/butler/registries/remote.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@
__all__ = ("RemoteRegistry",)

import contextlib
import functools
from collections.abc import Iterable, Iterator, Mapping, Sequence
from typing import TYPE_CHECKING, Any

Expand Down Expand Up @@ -237,7 +236,6 @@ def registerCollection(
# Docstring inherited from lsst.daf.butler.registry.Registry
raise NotImplementedError()

@functools.lru_cache
def getCollectionType(self, name: str) -> CollectionType:
# Docstring inherited from lsst.daf.butler.registry.Registry
# This could use a local cache since collection types won't
Expand Down

0 comments on commit 66b190f

Please sign in to comment.