-
Notifications
You must be signed in to change notification settings - Fork 915
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DataCatalog]: Provide public methods to modify catalog #3930
Comments
Adding a few more examples:
There's general agreement that we don't necessarily want to make all mutations of the catalog easy (like crazy injection of datasets in the middle of the lifecycle) but maybe there's more ways we can open up the collection of datasets just before the catalog is first instantiated for the rest of the run. For interactive use on the other hand, building the |
In the new catalog - The new catalog is partially mutable as it supports a setter which allows adding new or replacing existing datasets. We also decided with the team to not make catalog fully mutable. The datasets property remained private so as not to encourage behaviour when users configure the catalog via modifying the datasets dictionary. For the same reason It is also possible to modify the existing datasets in place as To see the full |
Description
Plugin developers and advanced users face limitations due to the absence of public methods for modifying the catalog datasets, and injecting dynamic behaviour or configuration parameters on the fly during pipeline execution. Although these limitations are made intentionally by not providing corresponding public APIs users bypass them by using private APIs.
We propose to:
DataCatalog
immutable.Relates to #2728
Context
https://github.com/Galileo-Galilei/kedro-mlflow/blob/64b8e94e1dafa02d979e7753dab9b9dfd4d7341c/kedro_mlflow/framework/hooks/mlflow_hook.py#L145
https://github.com/getindata/kedro-azureml/blob/d5c2011c7ed7fdc03235bf2bd6701f1901d1139c/kedro_azureml/hooks.py#L20
The text was updated successfully, but these errors were encountered: