-
Notifications
You must be signed in to change notification settings - Fork 30
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 - Issues with timestamptz data type #55
Comments
@juanbriones thanks so much for opening this issue and for chatting with myself and the team to help identify the solution to this Redshift bug! 🎉 💯 I was able to apply a bugfix in a working branch. Would you be able to swap your zendesk package dependency in your packages:
- git: https://github.com/fivetran/dbt_zendesk.git
revision: bugfix/redshift-timestamps
warn-unpinned: false Let me know if this then works. If it does, we can move forward with merging this into the next release! |
Hi Joe, how are you?
I will test it and let you know if I have any issues.
Thank you!
Best regards,
El lun, 13 dic 2021 a las 13:21, Joe Markiewicz ***@***.***>)
escribió:
… @juanbriones <https://github.com/juanbriones> thanks so much for opening
this issue and for chatting with myself and the team to help identify the
solution to this Redshift bug! 🎉 💯
I was able to apply a bugfix in a working branch. Would you be able to
swap your zendesk package dependency in your packages.yml for the below
dependency:
packages:
- git: https://github.com/fivetran/dbt_zendesk.git
revision: bugfix/redshift-timestamps
warn-unpinned: false
Let me know if this then works. If it does, we can move forward with
merging this into the next release!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#55 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA7Q2YEQ5FFLZVNXPKPNKUTUQYMRBANCNFSM5JPS6Y2Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
*Juan Francisco Briones*
|
Hi @juanbriones this fix is now live in the current version of I will be closing this issue as the current version of the package addressed the issue. Thank you again for all your help in assessing what this issue was, and how to solve it. We are extremely grateful for your contribution to this package and the community 😄 Feel free to open another issue if you have any other questions! |
Are you a current Fivetran customer?
Juan Francisco Briones, Data Engineer, Hopin.
Describe the bug
Apparently some fields from the original Zendesk data are in
timestamp with timezone (timestamptz)
and that makes thedbt_utils.datediff
anddbt_utils.dateadd
functions break.Steps to reproduce
Just run the
dbt_zendesk
package with date fields in timestamptz.Expected behavior
You will get the following error in multiple models:
Project variables configuration
Package Version
Warehouse
Additional context
I've already had a conversation with Joseph Markiewicz, Renee Li and Sheri Nguyen from Fivetran. They are aware of the issue and we solved it together casting the fields to
timestamp without time zone
.Please indicate the level of urgency
Medium priority. We need this information to create a dashboard for Customer Support team.
Are you interested in contributing to this package?
Joseph from Fivetran can help us as well.
The text was updated successfully, but these errors were encountered: