-
Notifications
You must be signed in to change notification settings - Fork 177
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
[Bug]: BigQuery profile target 'dev' invalid: 'schema' is a required property #1031
Comments
@pankajastro I think this bug relates to this change: #1017 It seems we are not adding
|
I wonder if it is a specific version of It seems that the issue with the 'schema' in profiles was introduced by the changes made in PR #1017, which was a continuation of the work done in PR #839. These PRs assumed that the BQ profile should not have to define the dataset and that this value could come from other dbt project files. @oliverrmaa @pankajastro , could you help us confirm the original use case and scenario so we can try to accommodate both cases? The While we don't release the fix, if you want to use 1.4.2, BQ users will have to add the following to their
|
In PR #1017, we attempted to remove `dataset` from the required fields list for the BigQuery profile. However, we realised that this is failing BiqQuery dbt operations as it indeed is a required field. Hence, bring back the same as a required field. This is also necessary for building the mock profile where we construct the profile by taking in consideration only the required fields. closes: #1031
In PR #1017, we attempted to remove `dataset` from the required fields list for the BigQuery profile. However, we realised that this is failing BiqQuery dbt operations as it indeed is a required field. Hence, bring back the same as a required field. This is also necessary for building the mock profile where we construct the profile by taking in consideration only the required fields. Closes: #1031
In PR #1017, we attempted to remove `dataset` from the required fields list for the BigQuery profile. However, we realised that this is failing BiqQuery dbt operations as it indeed is a required field. Hence, bring back the same as a required field. This is also necessary for building the mock profile where we construct the profile by taking in consideration only the required fields. Closes: #1031 (cherry picked from commit 803776a)
…omer#1033) In PR astronomer#1017, we attempted to remove `dataset` from the required fields list for the BigQuery profile. However, we realised that this is failing BiqQuery dbt operations as it indeed is a required field. Hence, bring back the same as a required field. This is also necessary for building the mock profile where we construct the profile by taking in consideration only the required fields. Closes: astronomer#1031
Astronomer Cosmos Version
Other Astronomer Cosmos version (please specify below)
If "Other Astronomer Cosmos version" selected, which one?
1.4.2
dbt-core version
1.5.0
Versions of dbt adapters
No response
LoadMode
AUTOMATIC
ExecutionMode
LOCAL
InvocationMode
None
airflow version
2.9.0
Operating System
Linux
If a you think it's an UI issue, what browsers are you seeing the problem on?
No response
Deployment
Official Apache Airflow Helm Chart
Deployment details
No response
What happened?
my unit tests detected some issues in v1.4.2 that did not occur in v1.4.1 or v1.4.0
from CI:
Relevant log output
Error Log:
From local container
The text was updated successfully, but these errors were encountered: