Skip to content

chore: Refactoring to fix mypy linting #3947

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

Closed
bushwhackr wants to merge 13 commits into from
Closed

chore: Refactoring to fix mypy linting #3947

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
68d18d2
allowing follow-imports + fixing on_demand_feature_view.py
bushwhackr Feb 11, 2024
73bba2c
creating base type `SnowflakeStoreConfig` to fix circular imports
bushwhackr Feb 11, 2024
d80acd8
fixing circular imports
bushwhackr Feb 11, 2024
3ecff66
converting create_data_source to take in arbitrary arguments
bushwhackr Feb 11, 2024
39fb45b
lint fix on base function `create_online_store`
bushwhackr Feb 11, 2024
83818e0
lint fix on ingest_stream_feature_view
bushwhackr Feb 11, 2024
2fa14b7
lint fixes in configuration
bushwhackr Feb 11, 2024
a96c266
isort linting
bushwhackr Feb 11, 2024
f0f7fec
mypy checking of all files in sdk/python folder
bushwhackr Feb 11, 2024
28010b7
checking only feast directory for mypy errors
bushwhackr Feb 11, 2024
5dbe13d
allow extras for FeastConfigBaseModel
bushwhackr Feb 13, 2024
004949d
fixing tests
bushwhackr Feb 13, 2024
2696cc3
adding extra field for online store config
bushwhackr Feb 13, 2024
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
lint fixes in configuration 
Signed-off-by: Chester Ong <chester.ong.ch@gmail.com>
  • Loading branch information
@bushwhackr
bushwhackr committed Feb 11, 2024
commit 2fa14b7ea3fef74ac899ac0cf86f7fdf8a642702
15 changes: 6 additions & 9 deletions sdk/python/tests/integration/feature_repos/repo_configuration.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,14 +99,14 @@
"host": os.getenv("ROCKSET_APISERVER", "api.rs2.usw2.rockset.com"),
}

OFFLINE_STORE_TO_PROVIDER_CONFIG: Dict[str, DataSourceCreator] = {
OFFLINE_STORE_TO_PROVIDER_CONFIG = {
"file": ("local", FileDataSourceCreator),
"bigquery": ("gcp", BigQueryDataSourceCreator),
"redshift": ("aws", RedshiftDataSourceCreator),
"snowflake": ("aws", SnowflakeDataSourceCreator),
}

AVAILABLE_OFFLINE_STORES: List[Tuple[str, Type[DataSourceCreator]]] = [
AVAILABLE_OFFLINE_STORES: List[Tuple[str, Any]] = [
("local", FileDataSourceCreator),
]

Expand All @@ -118,13 +118,10 @@

# Only configure Cloud DWH if running full integration tests
if os.getenv("FEAST_IS_LOCAL_TEST", "False") != "True":
AVAILABLE_OFFLINE_STORES.extend(
[
("gcp", BigQueryDataSourceCreator),
("aws", RedshiftDataSourceCreator),
("aws", SnowflakeDataSourceCreator),
]
)

AVAILABLE_OFFLINE_STORES.append(("gcp", BigQueryDataSourceCreator))
AVAILABLE_OFFLINE_STORES.append(("aws", RedshiftDataSourceCreator))
AVAILABLE_OFFLINE_STORES.append(("aws", SnowflakeDataSourceCreator))

AVAILABLE_ONLINE_STORES["redis"] = (REDIS_CONFIG, None)
AVAILABLE_ONLINE_STORES["dynamodb"] = (DYNAMO_CONFIG, None)
Expand Down