Doug Beatty
Doug Beatty
@Fleid ☝️ this is the ticket that we were discussing earlier today about inconsistent timestamp data types across adapters.
Awesome to see your thinking on this @jtcohen6 ! Agreed that inconsistency is unavoidable unless we break things. Also agreed that it is better to preserve that inconsistency rather than...
@timle2 I think branches that start with `feature/` have a special meaning for us. Can you rename it to something else (like `timle2/dbt-deps-tarball`) and see if it resolves the issue...
@timle2 Something for you to do: 1. Add this as the first line of your PR description: - `resolves #4205` - The ["resolves" keyword](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword) will link the PR to the...
@Olaktal Thanks for opening! Looking into this a bit (but without testing it myself), this looks like a bug to me. It would affect Redshift Let's take a quick look...
@jtcohen6 Thanks for expanding the context and correcting the key details! Logically, the balanced approach makes sense (but I'm guessing would be a bit tricky in implementation). The suggestion to...
Thank you for pointing me to that test @ChenyuLInx ! 👍 I will give it a shot and let you know how it goes.
@leahwicz Yeah, it would be awesome to get this into the 1.3 RC! @jtcohen6 would you be willing to give this a look? Specifically, how would we tell if it...
If https://github.com/dbt-labs/dbt-core/pull/5717 looks good to you, let's go with that one! > What's the difference between these two? These two pull requests were different originally, but now I've aligned them....
Closing in favor of #5717