Skip to content

Commit 3e259e7

Browse files
committed
refactor: update import paths for Catalog and CatalogV1 to reflect new schema structure
1 parent bd9bca6 commit 3e259e7

File tree

7 files changed

+25
-24
lines changed

7 files changed

+25
-24
lines changed

src/datapilot/core/platforms/dbt/executor.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
from datapilot.core.platforms.dbt.insights import INSIGHTS
1616
from datapilot.core.platforms.dbt.insights.schema import DBTInsightResult
1717
from datapilot.core.platforms.dbt.insights.schema import DBTModelInsightResponse
18-
from datapilot.core.platforms.dbt.schemas.manifest import Catalog
18+
from datapilot.core.platforms.dbt.schemas.catalog import Catalog
1919
from datapilot.core.platforms.dbt.schemas.manifest import Manifest
2020
from datapilot.core.platforms.dbt.utils import get_models
2121
from datapilot.utils.formatting.utils import RED

src/datapilot/core/platforms/dbt/factory.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
from dbt_artifacts_parser.parsers.manifest.manifest_v11 import ManifestV11
44
from dbt_artifacts_parser.parsers.manifest.manifest_v12 import ManifestV12
55

6-
from datapilot.core.platforms.dbt.schemas.manifest import Catalog
7-
from datapilot.core.platforms.dbt.schemas.manifest import CatalogV1
6+
from datapilot.core.platforms.dbt.schemas.catalog import Catalog
7+
from datapilot.core.platforms.dbt.schemas.catalog import CatalogV1
88
from datapilot.core.platforms.dbt.schemas.manifest import Manifest
99
from datapilot.core.platforms.dbt.wrappers.catalog.v1.wrapper import CatalogV1Wrapper
1010
from datapilot.core.platforms.dbt.wrappers.manifest.v10.wrapper import ManifestV10Wrapper

src/datapilot/core/platforms/dbt/schemas/catalog.py

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
from typing import Optional
66
from typing import Union
77

8+
from dbt_artifacts_parser.parsers.catalog.catalog_v1 import CatalogV1 as BaseCatalogV1
9+
from dbt_artifacts_parser.parsers.catalog.catalog_v1 import Metadata as BaseMetadata
810
from pydantic.main import BaseModel
911

1012

@@ -52,3 +54,19 @@ class AltimateCatalogCatalogV1(BaseModel):
5254
nodes: Dict[str, AltimateCatalogTable]
5355
sources: Dict[str, AltimateCatalogTable]
5456
errors: Optional[Optional[List[str]]] = None
57+
58+
59+
# Custom classes to handle extra fields in newer dbt versions
60+
class Metadata(BaseMetadata):
61+
class Config:
62+
extra = "allow" # Allow extra fields in metadata
63+
64+
65+
class CatalogV1(BaseCatalogV1):
66+
metadata: Metadata # Use our custom metadata class
67+
68+
class Config:
69+
extra = "allow" # Allow extra fields
70+
71+
72+
Catalog = CatalogV1

src/datapilot/core/platforms/dbt/schemas/manifest.py

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,6 @@
55
from typing import Optional
66
from typing import Union
77

8-
from dbt_artifacts_parser.parsers.catalog.catalog_v1 import CatalogV1 as BaseCatalogV1
9-
from dbt_artifacts_parser.parsers.catalog.catalog_v1 import Metadata as BaseMetadata
108
from dbt_artifacts_parser.parsers.manifest.manifest_v1 import ManifestV1
119
from dbt_artifacts_parser.parsers.manifest.manifest_v2 import ManifestV2
1210
from dbt_artifacts_parser.parsers.manifest.manifest_v3 import ManifestV3
@@ -45,21 +43,6 @@ class DBTVersion(BaseModel):
4543
]
4644

4745

48-
class Metadata(BaseMetadata):
49-
class Config:
50-
extra = "allow" # Allow extra fields in metadata
51-
52-
53-
class CatalogV1(BaseCatalogV1):
54-
metadata: Metadata # Use our custom metadata class
55-
56-
class Config:
57-
extra = "allow" # Allow extra fields
58-
59-
60-
Catalog = CatalogV1
61-
62-
6346
class AltimateDocs(BaseModel):
6447
show: Optional[bool] = True
6548
node_color: Optional[Optional[str]] = None

src/datapilot/core/platforms/dbt/utils.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,12 +17,12 @@
1717
from datapilot.core.platforms.dbt.constants import STAGING
1818
from datapilot.core.platforms.dbt.exceptions import AltimateInvalidManifestError
1919
from datapilot.core.platforms.dbt.factory import DBTFactory
20+
from datapilot.core.platforms.dbt.schemas.catalog import Catalog
21+
from datapilot.core.platforms.dbt.schemas.catalog import CatalogV1
2022
from datapilot.core.platforms.dbt.schemas.manifest import AltimateManifestExposureNode
2123
from datapilot.core.platforms.dbt.schemas.manifest import AltimateManifestNode
2224
from datapilot.core.platforms.dbt.schemas.manifest import AltimateManifestSourceNode
2325
from datapilot.core.platforms.dbt.schemas.manifest import AltimateManifestTestNode
24-
from datapilot.core.platforms.dbt.schemas.manifest import Catalog
25-
from datapilot.core.platforms.dbt.schemas.manifest import CatalogV1
2626
from datapilot.core.platforms.dbt.schemas.manifest import Manifest
2727
from datapilot.exceptions.exceptions import AltimateFileNotFoundError
2828
from datapilot.exceptions.exceptions import AltimateInvalidJSONError

src/datapilot/core/platforms/dbt/wrappers/catalog/v1/wrapper.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
from datapilot.core.platforms.dbt.schemas.manifest import CatalogV1
1+
from datapilot.core.platforms.dbt.schemas.catalog import CatalogV1
22
from datapilot.core.platforms.dbt.wrappers.catalog.wrapper import BaseCatalogWrapper
33

44

src/datapilot/utils/utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
from dbt_artifacts_parser.parser import parse_manifest
1313

1414
from datapilot.config.config import load_config
15-
from datapilot.core.platforms.dbt.schemas.manifest import CatalogV1
15+
from datapilot.core.platforms.dbt.schemas.catalog import CatalogV1
1616
from datapilot.schemas.nodes import ModelNode
1717
from datapilot.schemas.nodes import SourceNode
1818

0 commit comments

Comments
 (0)