dblift 3.2.0__tar.gz → 3.2.1__tar.gz

{dblift-3.2.0 → dblift-3.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dblift
-Version: 3.2.0
+Version: 3.2.1
 Summary: A database migration tool that supports multiple databases with native Python drivers
 Classifier: Development Status :: 5 - Production/Stable
 Classifier: Intended Audience :: Developers

{dblift-3.2.0 → dblift-3.2.1}/core/introspection/base_introspector.py

@@ -387,19 +387,21 @@ class BaseIntrospector:
         # Delegate to index extractor
         return self._get_index_extractor().get_indexes(schema, table)
 
-    def get_all_indexes(self, schema: str) -> List[Index]:
+    def get_all_indexes(self, schema: str) -> Optional[List[Index]]:
         """
         Get all indexes for an entire schema in a single bulk query.
 
-        Delegates to IndexExtractor.get_all_indexes. Returns an empty list
-        if the dialect does not support bulk index retrieval.
+        Delegates to IndexExtractor.get_all_indexes and propagates its
+        three-way answer unchanged.
 
         Args:
             schema: Schema name
 
         Returns:
-            List of Index objects for the entire schema, or [] if the dialect
-            does not support bulk retrieval.
+            ``None`` if the dialect cannot answer in bulk, meaning the caller
+            must fall back to per-table retrieval. ``[]`` if the bulk query ran
+            and the schema genuinely has no indexes. Otherwise the list of
+            Index objects for the entire schema.
         """
         return self._get_index_extractor().get_all_indexes(schema)
 

{dblift-3.2.0 → dblift-3.2.1}/core/introspection/extractors/index_extractor.py

@@ -1,6 +1,6 @@
 """Index extraction from plugin-owned vendor metadata queries."""
 
-from typing import Any, Dict, List
+from typing import Any, Dict, List, Optional
 
 from core.introspection.extractors.base_extractor import BaseExtractor
 from core.sql_model.index import Index
@@ -42,7 +42,7 @@ class IndexExtractor(BaseExtractor):
         # Convert to Index objects
         return self._build_index_objects(schema, table, indexes_data)
 
-    def get_all_indexes(self, schema: str) -> List[Index]:
+    def get_all_indexes(self, schema: str) -> Optional[List[Index]]:
         """
         Get all indexes for an entire schema.
 
@@ -52,17 +52,25 @@ class IndexExtractor(BaseExtractor):
             schema: Schema name
 
         Returns:
-            List of Index objects for the entire schema
+            ``None`` if bulk retrieval is unavailable — either no vendor
+            queries exist for the dialect, or its bulk index query declines —
+            meaning the caller must ask table by table.  ``[]`` if the bulk
+            query ran and the schema genuinely has no indexes.  Otherwise the
+            list of Index objects for the entire schema.
         """
+        # Unlike ColumnExtractor and TableExtractor.get_tables, which raise
+        # RuntimeError when their query is unavailable, this refuses with
+        # None: those two have no fallback, so refusing is the only honest
+        # option, whereas a working per-table path exists here and None routes
+        # the caller to it.  Raising would break callers that legitimately
+        # have that fallback.
         if not self.vendor_queries:
-            return []
+            return None
 
         self.ensure_metadata()
         query_result = self.vendor_queries.get_all_indexes_query(schema)
         if query_result is None:
-            return (
-                []
-            )  # no bulk query; empty for this test path (per-table fallback would be used by callers)
+            return None
 
         query, params = query_result
         rows = self.provider.query_executor.execute_query(self.connection, query, params)

{dblift-3.2.0 → dblift-3.2.1}/core/seams/sql_generators.py

@@ -35,7 +35,7 @@ def attach_registered_sql_generators() -> None:
 def clear_sql_generator_registrars() -> None:
     """Test hook."""
     global _bootstrapped
-    import core.seams.feature_loading as feature_loading
+    from core.seams import feature_loading
 
     _registrars.clear()
     _bootstrapped = False

{dblift-3.2.0 → dblift-3.2.1}/core/sql_model/view.py

@@ -298,7 +298,7 @@ class View(SqlObject):
         Returns:
             Dictionary with view attributes
         """
-        result = {
+        result: Dict[str, Any] = {
             "name": self.name,
             "schema": self.schema,
             "object_type": self.object_type.value,

{dblift-3.2.0 → dblift-3.2.1}/db/plugins/cosmosdb/cosmosdb/history_manager.py

@@ -419,7 +419,7 @@ class CosmosDbHistoryManager(DocumentHistoryManager):
         # row: a point delete addressed by the wrong partition key returns 404,
         # which would read as "already deleted" and remove nothing.
         query = "SELECT c.id, c.script, c.success, c.version FROM c WHERE c.script = @script"
-        parameters = [{"name": "@script", "value": script_name}]
+        parameters: List[Dict[str, object]] = [{"name": "@script", "value": script_name}]
         try:
             failed_docs = [
                 doc

{dblift-3.2.0 → dblift-3.2.1}/db/plugins/duckdb/parser/duckdb_regex_parser.py

@@ -5,7 +5,7 @@ so statement splitting only needs to respect string literals and comments
 around the ``;`` separator.
 """
 
-from typing import Any, Dict, List, Optional
+from typing import Dict, List, Optional
 
 from core.sql_model.base import SqlObject, SqlObjectType
 from core.sql_parser.enhanced_regex_parser import EnhancedRegexParser

{dblift-3.2.0 → dblift-3.2.1}/db/plugins/redshift/sqlalchemy_url.py

@@ -1,6 +1,6 @@
 """SQLAlchemy URL construction for the Redshift plugin."""
 
-from typing import Any, Dict, cast
+from typing import Any, Dict
 
 from sqlalchemy.engine import URL, make_url
 
@@ -33,18 +33,15 @@ def build_sqlalchemy_url(database_config: Any) -> str:
             query = _query_mapping(database_config, url.query)
             if query != dict(url.query):
                 url = url.set(query=query)
-            return cast(str, url.render_as_string(hide_password=False))
+            return url.render_as_string(hide_password=False)
         raise ValueError("Redshift native connections require a SQLAlchemy URL")
 
-    return cast(
-        str,
-        URL.create(
-            "redshift+redshift_connector",
-            username=getattr(database_config, "username", None) or None,
-            password=getattr(database_config, "password", None) or None,
-            host=getattr(database_config, "host", None) or "localhost",
-            port=getattr(database_config, "port", None),
-            database=getattr(database_config, "database", None),
-            query=_query_mapping(database_config),
-        ).render_as_string(hide_password=False),
-    )
+    return URL.create(
+        "redshift+redshift_connector",
+        username=getattr(database_config, "username", None) or None,
+        password=getattr(database_config, "password", None) or None,
+        host=getattr(database_config, "host", None) or "localhost",
+        port=getattr(database_config, "port", None),
+        database=getattr(database_config, "database", None),
+        query=_query_mapping(database_config),
+    ).render_as_string(hide_password=False)

{dblift-3.2.0 → dblift-3.2.1}/db/plugins/sqlite/provider.py

@@ -432,17 +432,6 @@ class SQLiteProvider(NativeProvider):
             connection, schema, create_schema, table_name
         )
 
-    def supports_snapshots(self) -> bool:
-        """SQLite does not own schema snapshot persistence."""
-        return False
-
-    def create_snapshot_table_if_not_exists(
-        self, schema: str, table_name: str = "dblift_schema_snapshots"
-    ) -> None:
-        """Satisfy BaseProvider while leaving snapshot storage unsupported."""
-        _ = (schema, table_name)
-        raise NotImplementedError("SQLite provider does not support schema snapshot persistence")
-
     def is_connected(self) -> bool:
         """Return True when an active sqlite3 connection is held.
 

{dblift-3.2.0 → dblift-3.2.1}/dblift.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dblift
-Version: 3.2.0
+Version: 3.2.1
 Summary: A database migration tool that supports multiple databases with native Python drivers
 Classifier: Development Status :: 5 - Production/Stable
 Classifier: Intended Audience :: Developers

{dblift-3.2.0 → dblift-3.2.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "dblift"
-version = "3.2.0"
+version = "3.2.1"
 description = "A database migration tool that supports multiple databases with native Python drivers"
 readme = "README.md"
 requires-python = ">=3.11"