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.

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

Fix the bug introduced by this commit 350d5443a19e0c8db8aec0eae74624e2235c7d3f #20871

Closed
wants to merge 1 commit into from

Conversation

grishick
Copy link
Contributor

Destination BigQuery denormalized does not support normalization. The change that added normalization definitions to actor definitions table (350d544) broke DATs for bigquery denormalized by erroneously adding normalization config for it. This change removes normalization config from destination-bigquery-denormalized in destination_definitions.yaml

@grishick
Copy link
Contributor Author

grishick commented Dec 23, 2022

/test connector=connectors/destination-bigquery-denormalized

🕑 connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/3767833460
❌ connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/3767833460
🐛 https://gradle.com/s/elktamlvzmuxg

Build Failed

Test summary info:

Could not find result summary

@grishick grishick temporarily deployed to more-secrets December 23, 2022 19:30 — with GitHub Actions Inactive
@grishick
Copy link
Contributor Author

Closing in favor of #20640

@grishick grishick closed this Dec 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant