Skip to content

Commit

Permalink
fix(ingest/kafka): fix ResourceType import error for confluent_kafka<…
Browse files Browse the repository at this point in the history
  • Loading branch information
mayurinehate authored and Eric Yomi committed Jan 18, 2023
1 parent 8b2d8d5 commit 3f5e1f8
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions metadata-ingestion/src/datahub/ingestion/source/kafka.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
AdminClient,
ConfigEntry,
ConfigResource,
ResourceType,
TopicMetadata,
)

Expand Down Expand Up @@ -404,11 +403,10 @@ def fetch_extra_topic_details(self, topics: List[str]) -> Dict[str, dict]:

def fetch_topic_configurations(self, topics: List[str]) -> Dict[str, dict]:
logger.info("Fetching config details for all topics")

configs: Dict[
ConfigResource, concurrent.futures.Future
] = self.admin_client.describe_configs(
resources=[ConfigResource(ResourceType.TOPIC, t) for t in topics],
resources=[ConfigResource(ConfigResource.Type.TOPIC, t) for t in topics],
request_timeout=self.source_config.connection.client_timeout_seconds,
)
logger.debug("Waiting for config details futures to complete")
Expand Down

0 comments on commit 3f5e1f8

Please sign in to comment.