Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(snowflake): set is_temp_table and is_allowed_table function for SqlParsingAggregator in SnowflakeV2Source #12438

Merged
merged 4 commits into from
Jan 28, 2025

Merge branch 'master' into feature/ing-804-feat-snowflake-is-temp-table

0dda7c5
Select commit
Loading
Failed to load commit list.
Merged

feat(snowflake): set is_temp_table and is_allowed_table function for SqlParsingAggregator in SnowflakeV2Source #12438

Merge branch 'master' into feature/ing-804-feat-snowflake-is-temp-table
0dda7c5
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jan 28, 2025 in 1s

78.57% of diff hit (target 90.00%)

View this Pull Request on Codecov

78.57% of diff hit (target 90.00%)

Annotations

Check warning on line 458 in metadata-ingestion/src/datahub/ingestion/source/snowflake/snowflake_v2.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/snowflake/snowflake_v2.py#L458 

Added line #L458 was not covered by tests

Check warning on line 469 in metadata-ingestion/src/datahub/ingestion/source/snowflake/snowflake_v2.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/snowflake/snowflake_v2.py#L469 

Added line #L469 was not covered by tests

Check warning on line 475 in metadata-ingestion/src/datahub/ingestion/source/snowflake/snowflake_v2.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/snowflake/snowflake_v2.py#L475 

Added line #L475 was not covered by tests
View more details on Codecov