Feedback 1

Remove streams
Move fixtures to amundsen_common/tests
Stat as additional class, deprecate Statistics

Signed-off-by: Joshua Hoskins <hoskins@squareup.com>

amundsen_common/models/table.py

@@ -60,6 +60,26 @@ class Meta:
         register_as_scheme = True
 
 
+@attr.s(auto_attribs=True, kw_only=True)
+class Statistics:
+    """
+    DEPRECATED. Use Stat
+    """
+    stat_type: str
+    stat_val: Optional[str] = None
+    start_epoch: Optional[int] = None
+    end_epoch: Optional[int] = None
+
+
+class StatisticsSchema(AttrsSchema):
+    """
+    DEPRECATED. Use StatSchema
+    """
+    class Meta:
+        target = Statistics
+        register_as_scheme = True
+
+
 @attr.s(auto_attribs=True, kw_only=True)
 class Stat:
     stat_type: str

amundsen_common/tests/__init__.py

@@ -0,0 +1,2 @@
+# Copyright Contributors to the Amundsen project.
+# SPDX-License-Identifier: Apache-2.0

amundsen_common/fixtures.py → amundsen_common/tests/fixtures.py

@@ -11,6 +11,9 @@
 
 
 class Fixtures:
+    """
+    These fixtures are useful for creating test objects. For an example usage, check out tests/tests/test_fixtures.py
+    """
     counter = 1000
 
     @staticmethod