Skip to content

Commit 7c3bd2e

Browse files
marti-jorda-rocazseta
authored andcommitted
fix: Bytewax materialization engine fails when loading feature_store.yaml (feast-dev#3912)
* bytewax materialization loads yaml config correctly Signed-off-by: marti-jorda-roca <mjorda98@gmail.com> * added postgres dependency for SQL registries Signed-off-by: marti-jorda-roca <mjorda98@gmail.com> --------- Signed-off-by: marti-jorda-roca <mjorda98@gmail.com> Signed-off-by: Attila Toth <hello@attilatoth.dev>
1 parent 8e851d9 commit 7c3bd2e

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

sdk/python/feast/infra/materialization/contrib/bytewax/Dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -25,5 +25,5 @@ COPY README.md README.md
2525
# git dir to infer the version of feast we're installing.
2626
# https://github.com/pypa/setuptools_scm#usage-from-docker
2727
# I think it also assumes that this dockerfile is being built from the root of the directory.
28-
RUN --mount=source=.git,target=.git,type=bind pip3 install --no-cache-dir '.[aws,gcp,bytewax,snowflake]'
28+
RUN --mount=source=.git,target=.git,type=bind pip3 install --no-cache-dir '.[aws,gcp,bytewax,snowflake,postgres]'
2929

sdk/python/feast/infra/materialization/contrib/bytewax/dataflow.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,10 @@
1212
logging.basicConfig(level=logging.INFO)
1313

1414
with open("/var/feast/feature_store.yaml") as f:
15-
feast_config = yaml.safe_load(f)
15+
feast_config = yaml.load(f, Loader=yaml.Loader)
1616

1717
with open("/var/feast/bytewax_materialization_config.yaml") as b:
18-
bytewax_config = yaml.safe_load(b)
18+
bytewax_config = yaml.load(b, Loader=yaml.Loader)
1919

2020
config = RepoConfig(**feast_config)
2121
store = FeatureStore(config=config)

0 commit comments

Comments
 (0)