Skip to content

fix(filter_visibility): address fourth-pass review findings

96af826
Select commit
Loading
Failed to load commit list.
Open

feat(ingestion): cross-connector filter visibility report #28355

fix(filter_visibility): address fourth-pass review findings
96af826
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ingestion] SonarCloud Code Analysis succeeded May 22, 2026 in 3m 57s

Annotations

Check warning on line 862 in ingestion/src/metadata/ingestion/source/database/database_service.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Use "set.update()" instead of a for-loop with "add()".

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGr668hX41I62CIk&open=AZ5MmGr668hX41I62CIk&pullRequest=28355

Check failure on line 528 in ingestion/src/metadata/ingestion/source/database/snowflake/metadata.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 23 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGox68hX41I62CIh&open=AZ5MmGox68hX41I62CIh&pullRequest=28355

Check failure on line 633 in ingestion/src/metadata/ingestion/source/storage/storage_service.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGsP68hX41I62CIn&open=AZ5MmGsP68hX41I62CIn&pullRequest=28355

Check failure on line 361 in ingestion/src/metadata/ingestion/source/database/common_db_source.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 32 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGpY68hX41I62CIj&open=AZ5MmGpY68hX41I62CIj&pullRequest=28355

Check failure on line 211 in ingestion/src/metadata/ingestion/source/storage/storage_service.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 19 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGsP68hX41I62CIl&open=AZ5MmGsP68hX41I62CIl&pullRequest=28355

Check failure on line 220 in ingestion/src/metadata/ingestion/source/pipeline/pipeline_service.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGiW68hX41I62CIe&open=AZ5MmGiW68hX41I62CIe&pullRequest=28355

Check warning on line 533 in ingestion/src/metadata/ingestion/source/dashboard/dashboard_service.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Use a union type expression for this type hint.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGoT68hX41I62CIg&open=AZ5MmGoT68hX41I62CIg&pullRequest=28355

Check failure on line 92 in ingestion/src/metadata/ingestion/source/database/common_db_source.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Add an explicit default value to this optional field.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGpY68hX41I62CIi&open=AZ5MmGpY68hX41I62CIi&pullRequest=28355

Check warning on line 136 in ingestion/src/metadata/ingestion/source/mlmodel/sagemaker/metadata.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Add replacement fields or use a normal string instead of an f-string.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGtb68hX41I62CIp&open=AZ5MmGtb68hX41I62CIp&pullRequest=28355

Check failure on line 494 in ingestion/src/metadata/ingestion/source/storage/storage_service.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 31 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGsP68hX41I62CIm&open=AZ5MmGsP68hX41I62CIm&pullRequest=28355

Check warning on line 132 in ingestion/src/metadata/ingestion/source/mlmodel/sagemaker/metadata.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Add replacement fields or use a normal string instead of an f-string.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGtb68hX41I62CIo&open=AZ5MmGtb68hX41I62CIo&pullRequest=28355

Check warning on line 532 in ingestion/src/metadata/ingestion/source/dashboard/dashboard_service.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Use a union type expression for this type hint.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ5MmGoT68hX41I62CIf&open=AZ5MmGoT68hX41I62CIf&pullRequest=28355