Rename in comments

Signed-off-by: Achal Shah <achals@gmail.com>
feast-dev · feast-ci-bot · Jul 22, 2021 · Jul 6, 2021 · Jul 6, 2021 · Jul 6, 2021
commit c457f1ecdff8eaa68fd751808625f054542a80ae
@@ -26,10 +26,10 @@ class FeatureServiceNotFoundException(FeastObjectNotFoundException):
     def __init__(self, name, project=None):
         if project:
             super().__init__(
-                f"FeatureService {name} does not exist in project {project}"
+                f"Feature service {name} does not exist in project {project}"
             )
         else:
-            super().__init__(f"FeatureService {name} does not exist")
+            super().__init__(f"Feature service {name} does not exist")
 
 
 class FeatureViewNotFoundException(FeastObjectNotFoundException):

@@ -319,8 +319,8 @@ def get_historical_features(
                 column used to ensure point-in-time correctness. Either a Pandas DataFrame can be provided or a string
                 SQL query. The query must be of a format supported by the configured offline store (e.g., BigQuery)
             features: A list of features, that should be retrieved from the offline store.
-                Feature references are of the format "feature_view:feature", e.g., "customer_fv:daily_transactions",
-                or it can be an instance of a FeatureService object.
+                Either a list of string feature references can be provided or a FeatureService object.
+                Feature references are of the format "feature_view:feature", e.g., "customer_fv:daily_transactions".
             full_feature_names: A boolean that provides the option to add the feature view prefixes to the feature names,
                 changing them from the format "feature" to "feature_view__feature" (e.g., "daily_transactions" changes to
                 "customer_fv__daily_transactions"). By default, this value is set to False.
@@ -355,11 +355,9 @@ def get_historical_features(
             _feature_refs = _features
 
         all_feature_views = self._registry.list_feature_views(project=self.project)
-        print(f"All feature views: {all_feature_views}")
         feature_views = list(
             view for view, _ in _group_feature_refs(_feature_refs, all_feature_views)
         )
-        print(f"Selected feature views: {feature_views}")
 
         _validate_feature_refs(_feature_refs, full_feature_names)
 
@@ -717,8 +715,6 @@ def _group_feature_refs(
 ) -> List[Tuple[FeatureView, List[str]]]:
     """ Get list of feature views and corresponding feature names based on feature references"""
 
-    print(f"Input Features: {features}")
-
     # view name to view proto
     view_index = {view.name: view for view in all_feature_views}
 

@@ -126,10 +126,10 @@ def list_entities(self, project: str, allow_cache: bool = False) -> List[Entity]
 
     def apply_feature_service(self, feature_service: FeatureService, project: str):
         """
-        Registers a single entity with Feast
+        Registers a single Feature service with Feast
 
         Args:
-            feature_service: FeatureService that will be registered
+            feature_service: A Feature service that will be registered
             project: Feast project that this entity belongs to
         """
         feature_service_proto = feature_service.to_proto()
@@ -180,14 +180,14 @@ def get_feature_service(
         self, name: str, project: str, allow_cache: bool = False
     ) -> FeatureService:
         """
-        Retrieves a FeatureService.
+        Retrieves a Feature service.
 
         Args:
-            name: Name of FeatureService
-            project: Feast project that this FeatureService belongs to
+            name: Name of Feature service
+            project: Feast project that this Feature service belongs to
 
         Returns:
-            Returns either the specified FeatureService, or raises an exception if
+            Returns either the specified Feature service, or raises an exception if
             none is found
         """
         registry_proto = self._get_registry_proto(allow_cache=allow_cache)

@@ -119,13 +119,13 @@ def parse_repo(repo_root: Path) -> ParsedRepo:
 def apply_feature_services(registry: Registry, project: str, repo: ParsedRepo):
     from colorama import Fore, Style
 
-    # Determine which FeatureServices should be deleted.
+    # Determine which Feature services should be deleted.
     existing_feature_services = registry.list_feature_services(project)
     for feature_service in repo.feature_services:
         if feature_service in existing_feature_services:
             existing_feature_services.remove(feature_service)
 
-    # The remaining features services in the list should be deleted.
+    # The remaining Features services in the list should be deleted.
     for feature_service_to_delete in existing_feature_services:
         registry.delete_feature_service(feature_service_to_delete.name, project)
         click.echo(