Skip to content

fix: Don't prevent apply from running given duplicate empty names in data sources. Also fix repeated apply of Spark data source. #2415

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.

Sign up for GitHub

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

Merged
feast-ci-bot merged 15 commits into from
Mar 29, 2022
Merged

fix: Don't prevent apply from running given duplicate empty names in data sources. Also fix repeated apply of Spark data source. #2415

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
2db0c9a
fix: Print more warning statements on requirement for data sources to…
adchia Mar 17, 2022
2d4fc84
typo
adchia Mar 17, 2022
88a7434
typo
adchia Mar 17, 2022
ba97190
fix
adchia Mar 17, 2022
cdee46c
fix
adchia Mar 17, 2022
3a4b4cd
fix
adchia Mar 17, 2022
7add51a
More tests
adchia Mar 21, 2022
bc9d3ae
fix
adchia Mar 29, 2022
9d2c5f1
fix
adchia Mar 29, 2022
bae2e37
fix
adchia Mar 29, 2022
41ff4d9
fix
adchia Mar 29, 2022
70d4b8d
revert
adchia Mar 29, 2022
e17973f
fix
adchia Mar 29, 2022
0fe2df8
fix
adchia Mar 29, 2022
f854de4
fix
adchia Mar 29, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix 
Signed-off-by: Danny Chiao <danny@tecton.ai>
  • Loading branch information
@adchia
adchia committed Mar 29, 2022
commit bc9d3ae8d3910edd05ebdd4da84a686a36652049
2 changes: 1 addition & 1 deletion sdk/python/feast/go_server.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,10 @@
from subprocess import Popen
from typing import Any, Dict, List, Optional, Union

import grpc
from tenacity import retry, stop_after_attempt, stop_after_delay, wait_exponential

import feast
import grpc
from feast.errors import FeatureNameCollisionError, InvalidFeaturesParameterType
from feast.feature_service import FeatureService
from feast.flags_helper import is_test
Expand Down
4 changes: 3 additions & 1 deletion sdk/python/feast/registry.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,9 @@ def get_objects_from_registry(
registry: "Registry", project: str
) -> Dict["FeastObjectType", List[Any]]:
return {
FeastObjectType.DATA_SOURCE: sorted(registry.list_data_sources(project=project)),
FeastObjectType.DATA_SOURCE: sorted(
registry.list_data_sources(project=project)
),
FeastObjectType.ENTITY: registry.list_entities(project=project),
FeastObjectType.FEATURE_VIEW: registry.list_feature_views(project=project),
FeastObjectType.ON_DEMAND_FEATURE_VIEW: registry.list_on_demand_feature_views(
Expand Down
2 changes: 1 addition & 1 deletion sdk/python/feast/transformation_server.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@
import sys
from concurrent import futures

import grpc
import pyarrow as pa
from grpc_reflection.v1alpha import reflection

import grpc
from feast.errors import OnDemandFeatureViewNotFoundException
from feast.feature_store import FeatureStore
from feast.protos.feast.serving.TransformationService_pb2 import (
Expand Down