Skip to content

Commit

Permalink
docs(ingest/looker): mark platform instance as a supported capability (
Browse files Browse the repository at this point in the history
  • Loading branch information
hsheth2 authored Nov 30, 2023
1 parent c3499f8 commit efaf21d
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@

from datahub.configuration import ConfigModel
from datahub.configuration.common import AllowDenyPattern, ConfigurationError
from datahub.configuration.source_common import DatasetSourceConfigMixin, EnvConfigMixin
from datahub.configuration.source_common import (
EnvConfigMixin,
PlatformInstanceConfigMixin,
)
from datahub.configuration.validate_field_removal import pydantic_removed_field
from datahub.ingestion.source.looker.looker_lib_wrapper import LookerAPIConfig
from datahub.ingestion.source.state.stale_entity_removal_handler import (
Expand Down Expand Up @@ -98,7 +101,7 @@ class LookerViewNamingPattern(NamingPattern):
]


class LookerCommonConfig(DatasetSourceConfigMixin):
class LookerCommonConfig(EnvConfigMixin, PlatformInstanceConfigMixin):
explore_naming_pattern: LookerNamingPattern = pydantic.Field(
description=f"Pattern for providing dataset names to explores. {LookerNamingPattern.allowed_docstring()}",
default=LookerNamingPattern(pattern="{model}.explore.{name}"),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@
@support_status(SupportStatus.CERTIFIED)
@config_class(LookerDashboardSourceConfig)
@capability(SourceCapability.DESCRIPTIONS, "Enabled by default")
@capability(SourceCapability.PLATFORM_INSTANCE, "Not supported", supported=False)
@capability(SourceCapability.PLATFORM_INSTANCE, "Use the `platform_instance` field")
@capability(
SourceCapability.OWNERSHIP, "Enabled by default, configured using `extract_owners`"
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1455,7 +1455,7 @@ class LookerManifest:
@support_status(SupportStatus.CERTIFIED)
@capability(
SourceCapability.PLATFORM_INSTANCE,
"Supported using the `connection_to_platform_map`",
"Use the `platform_instance` and `connection_to_platform_map` fields",
)
@capability(SourceCapability.LINEAGE_COARSE, "Supported by default")
@capability(
Expand Down

0 comments on commit efaf21d

Please sign in to comment.