Skip to content

Commit

Permalink
Replace call of validate_put_parameters with isAcceptable for InMemor…
Browse files Browse the repository at this point in the history
…yDatastore.
  • Loading branch information
erykoff committed Aug 17, 2024
1 parent 0dc3e95 commit e0c53f9
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion python/lsst/daf/butler/datastores/inMemoryDatastore.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
from urllib.parse import urlencode

from lsst.daf.butler import DatasetId, DatasetRef, StorageClass
from lsst.daf.butler._exceptions import DatasetTypeNotSupportedError
from lsst.daf.butler.datastore import DatasetRefURIs, DatastoreConfig
from lsst.daf.butler.datastore.generic_base import GenericBaseDatastore, post_process_get
from lsst.daf.butler.datastore.record_data import DatastoreRecordData
Expand Down Expand Up @@ -407,7 +408,11 @@ def put(self, inMemoryDataset: Any, ref: DatasetRef) -> None:
if not delegate or not delegate.can_accept(inMemoryDataset):
inMemoryDataset = ref.datasetType.storageClass.coerce_type(inMemoryDataset)

self._validate_put_parameters(inMemoryDataset, ref)
if not self.constraints.isAcceptable(ref):
# Raise rather than use boolean return value.
raise DatasetTypeNotSupportedError(
f"Dataset {ref} has been rejected by this datastore via configuration."
)

self.datasets[ref.id] = inMemoryDataset
log.debug("Store %s in %s", ref, self.name)
Expand Down

0 comments on commit e0c53f9

Please sign in to comment.