Skip to content

Commit

Permalink
feat(ingest): Couchbase source lint fix update
Browse files Browse the repository at this point in the history
  • Loading branch information
mminichino committed Jan 23, 2025
1 parent d45700e commit 2cc5ba5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ def discard(_tuple: Tuple[str, ...], element: str) -> Tuple[str, ...]:


def construct_schema(
collection: Dict[str, Any]
collection: Dict[str, Any],
) -> Dict[Tuple[str, ...], SchemaDescription]:
"""
Construct JSON schema.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -352,9 +352,9 @@ def construct_schema_metadata(
)
# Add this information to the custom properties so user can know they are looking at downsampled schema
dataset_properties.customProperties["schema.downsampled"] = "True"
dataset_properties.customProperties[
"schema.totalFields"
] = f"{collection_schema_size}"
dataset_properties.customProperties["schema.totalFields"] = (

Check warning on line 355 in metadata-ingestion/src/datahub/ingestion/source/couchbase/couchbase_main.py

View check run for this annotation

Codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/couchbase/couchbase_main.py#L354-L355

Added lines #L354 - L355 were not covered by tests
f"{collection_schema_size}"
)

logger.debug(f"Size of collection fields = {len(collection_fields)}")
# append each schema field (sort so output is consistent)
Expand Down

0 comments on commit 2cc5ba5

Please sign in to comment.