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

Jump to bottom

Snowflake SP don't filter by warehouse #14018

Merged

pmbrull merged 1 commit into open-metadata:main from pmbrull:snowflake-no-wh-filter

Nov 17, 2023

Collaborator

pmbrull commented Nov 17, 2023

Describe your changes:

Since the service can live in one warehouse and the queries are executed in another

Type of change:

Checklist:

I have read the CONTRIBUTING document.
My PR title is Fixes <issue-number>: <short explanation>
I have commented on my code, particularly in hard-to-understand areas.
For JSON Schema changes: I updated the migration scripts or explained why it is not needed.


          Snowflake SP don't filter by warehouse

d0d4cee

pmbrull requested a review from a team as a code owner

November 17, 2023 12:01

github-actions bot added Ingestion safe to test labels

OnkarVO7 approved these changes

View reviewed changes

sonarqubecloud bot commented Nov 17, 2023

[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

pmbrull merged commit a8a1fd2 into open-metadata:main

12 checks passed

pmbrull deleted the snowflake-no-wh-filter branch

November 17, 2023 13:18

pmbrull added a commit that referenced this pull request


          Snowflake SP don't filter by warehouse (#14018)

d153412

pmbrull added a commit that referenced this pull request


          Snowflake SP don't filter by warehouse (#14018)

9fe4a2f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ingestion safe to test