Remove default list from the FeatureView constructor (feast-dev#1679)

achals · Mwad22 · commit 183a0b9dad67 · 2021-07-07T17:33:17.000-04:00
Signed-off-by: Achal Shah &lt;achals@gmail.com&gt;
Signed-off-by: Mwad22 &lt;51929507+Mwad22@users.noreply.github.com&gt;
diff --git a/sdk/python/feast/feature_view.py b/sdk/python/feast/feature_view.py
@@ -65,14 +65,16 @@ def __init__(
         input: DataSource,
         batch_source: Optional[DataSource] = None,
         stream_source: Optional[DataSource] = None,
-        features: List[Feature] = [],
+        features: List[Feature] = None,
         tags: Optional[Dict[str, str]] = None,
         online: bool = True,
     ):
         _input = input or batch_source
         assert _input is not None
 
-        cols = [entity for entity in entities] + [feat.name for feat in features]
+        _features = features or []
+
+        cols = [entity for entity in entities] + [feat.name for feat in _features]
         for col in cols:
             if _input.field_mapping is not None and col in _input.field_mapping.keys():
                 raise ValueError(
@@ -83,7 +85,7 @@ def __init__(
 
         self.name = name
         self.entities = entities
-        self.features = features
+        self.features = _features
         self.tags = tags if tags is not None else {}
 
         if isinstance(ttl, Duration):
