Skip to content
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

FeatureStore, FeatureView, Config, and BIgQuerySource classes for updated SDK #1364

Merged
merged 5 commits into from
Mar 6, 2021
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
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
37 changes: 37 additions & 0 deletions sdk/python/feast/big_query_source.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
# Copyright 2019 The Feast Authors
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# https://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
from typing import Dict, Optional


class BigQuerySource:
"""
Represents a BigQuery table reference or BigQuery query that returns a set of features.
"""

def __init__(
self,
table_ref: Optional[str],
event_timestamp_column: str,
created_timestamp_column: Optional[str],
field_mapping: Optional[Dict[str, str]],
query: Optional[str],
):
if (table_ref is None) != (query is None):
raise Exception("Exactly one of table_ref and query should be specified")
self.table_ref = table_ref
self.event_timestamp_column = event_timestamp_column
self.created_timestamp_column = created_timestamp_column
self.field_mapping = field_mapping
self.query = query
return
34 changes: 34 additions & 0 deletions sdk/python/feast/feature_store.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
# Copyright 2019 The Feast Authors
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# https://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
from typing import Optional

from feast.feature_store_config import Config


class FeatureStore:
woop marked this conversation as resolved.
Show resolved Hide resolved
"""
Fat client to interact with the Feast feature store.
"""

def __init__(
self, config_path: Optional[str], config: Optional[Config],
):
if config_path is None or config is None:
raise Exception("You cannot specify both config_path and config")
if config is not None:
self.config = config
elif config_path is not None:
self.config = Config.from_config_path(config_path)
else:
self.config = Config()
44 changes: 44 additions & 0 deletions sdk/python/feast/feature_store_config.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
# Copyright 2019 The Feast Authors
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# https://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
from typing import Optional

import yaml


class Config:
woop marked this conversation as resolved.
Show resolved Hide resolved
"""
Python representation of the FeatureStore config that the user can specify via a yaml file.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: If the user doesnt know what the yaml file is then this comment doesnt add value. How about Configuration object that contains all possible configuration options for a FeatureStore?

"""

def __init__(
self,
provider: Optional[str],
online_store: Optional[str],
metadata_store: Optional[str],
):
self.provider = provider if (provider is not None) else "local"
self.online_store = online_store if (online_store is not None) else "local"
self.metadata_store = (
metadata_store if (metadata_store is not None) else "./metadata_store"
)

@classmethod
def from_config_path(cls, config_path: str):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

from_path perhaps?

with open(config_path, "r") as f:
config_dict = yaml.safe_load(f)
return cls(
provider=config_dict.get("provider"),
online_store=config_dict.get("online_store"),
metadata_store=config_dict.get("metadata_store"),
)
46 changes: 46 additions & 0 deletions sdk/python/feast/feature_view.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
# Copyright 2019 The Feast Authors
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# https://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
from datetime import datetime
from typing import Dict, List

from feast.big_query_source import BigQuerySource
from feast.entity import Entity
from feast.feature import Feature


class FeatureView:
"""
Represents a collection of features that will be served online.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could in theory also have batch only users. How about A FeatureView defines a logical grouping of servable features?

"""

def __init__(
self,
name: str,
entities: List[Entity],
features: List[Feature],
tags: Dict[str, str],
ttl: str,
online: bool,
inputs: BigQuerySource,
feature_start_time: datetime,
):
self.name = name
self.entities = entities
self.features = features
self.tags = tags
self.ttl = ttl
self.online = online
self.inputs = inputs
self.feature_start_time = feature_start_time
return