Skip to content

Commit

Permalink
Merge pull request #34 from Ki-Insurance/fix-bugfix-assertion-error-m…
Browse files Browse the repository at this point in the history
…aterialisation

Skip the empty job instead of erroring out
  • Loading branch information
mateusz-ki authored Jul 1, 2024
2 parents 142cf33 + c67f0ce commit 9e15736
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion sdk/python/feast/infra/passthrough_provider.py
Original file line number Diff line number Diff line change
Expand Up @@ -282,7 +282,9 @@ def materialize_single_feature_view(
tqdm_builder=tqdm_builder,
)
jobs = self.batch_engine.materialize(registry, [task])
assert len(jobs) == 1
# Empty jobs list might happen when there is no new data to materialize. In that case, we would just skip the execution and move on to another view.
if len(jobs) == 0:
return
if jobs[0].status() == MaterializationJobStatus.ERROR and jobs[0].error():
e = jobs[0].error()
assert e
Expand Down

0 comments on commit 9e15736

Please sign in to comment.