fix 2 review comments

dlt/common/storages/live_schema_storage.py

@@ -63,9 +63,7 @@ def is_live_schema_committed(self, name: str) -> bool:
             raise SchemaNotFoundError(name, f"live-schema://{name}")
         try:
             stored_schema_json = self._load_schema_json(name)
-            return live_schema.stored_version_hash == cast(
-                str, stored_schema_json.get("version_hash")
-            )
+            return live_schema.version_hash == cast(str, stored_schema_json.get("version_hash"))
         except FileNotFoundError:
             return False
 

dlt/extract/extract.py

@@ -75,7 +75,8 @@ def choose_schema() -> Schema:
         """Except of explicitly passed schema, use a clone that will get discarded if extraction fails"""
         if schema:
             schema_ = schema
-        # if we have a default schema name, use it but do not collect any info from the existing schema
+        # TODO: We should start with a new schema of the same name here ideally, but many tests fail
+        # because of this. So some investigation is needed.
         elif pipeline.default_schema_name:
             schema_ = pipeline.schemas[pipeline.default_schema_name].clone()
         else: