-
Notifications
You must be signed in to change notification settings - Fork 38
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
[Feature] dbt compile does not work on new schema #83
Comments
We're hitting this problem when trying to use this model on dbt cloud - appears we are running before compiling as well and coming across this problem. We can't use Fivetran to run this quickbooks transformer, due to a separate bug with number precision in generated tables in our Snowflake destination. Is there an ETA for a fix on this? Thanks! |
Hey @josler thanks for commenting here and sorry to hear you are running into this issue. Before diving into your case, would you be able to add more detail to your comment here
Is this a rounding bug you are seeing in the package or directly with the source data synced by Fivetran? I mainly ask because if it is either I am not entirely sure transforming via dbt Cloud or Fivetran Transformations will resolve this issue. Let me know! Now to your initial question! We have seen this error occur when you are trying to run the package for the very first time (especially in some dbt Cloud envs) due to dbt_quickbooks/models/intermediate/int_quickbooks__general_ledger_date_spine.sql Lines 5 to 8 in 5fe28a4
The issue is that when running dbt compile, dbt is looking for the In other packages, we have avoided this error by changing the Unfortunately there is no ETA on this fix as I am unsure what the best path forward is at the moment. Do you have any ideas or solutions we could implement to solve this on your end? Once we identify a solution, the ETA should be fairly quick to patch this in an upcoming release. |
Hey, On the precision issue it's something that Fivetran support told me they're aware of and the team is looking into. I think it's related to how the source data was synced. In any case, for this issue I don't have enough expertise here to suggest a useful solution without thinking about it further. We couldn't get this transform running correctly on dbt cloud either so are exploring alternative solutions (not necessarily dbt-powered). Thanks! |
Is there an existing feature request for this?
Describe the Feature
Currently when attempting to run
dbt compile
without doing adbt run
previously you are hit with a compilation error in the date spine model. This is due to thequickbooks__general_ledger
model not being created yet and it is a dependency of the date spine.It would be preferable if the package can run
dbt compile
without needing adbt run
. This will help with CI/CD processes and ensure users will be able to validate the compiled code prior to execution.Describe alternatives you've considered
Running
dbt run
beforedbt compile
. This solves the issue, but is not the ideal solution.Are you interested in contributing this feature?
Anything else?
This may not be able to be fully addressed. However, it should be investigated and at the least documented for users.
The text was updated successfully, but these errors were encountered: