dbt-codegen icon indicating copy to clipboard operation
dbt-codegen copied to clipboard

Update Changelog

Open gwenwindflower opened this issue 1 year ago • 1 comments

gwenwindflower avatar Apr 17 '24 14:04 gwenwindflower

@gwenwindflower is it possible to get this merged with a release? To have https://github.com/dbt-labs/dbt-codegen/pull/168 supported without pinning to a git branch 😄 . I'm happy to help with anything needed to get it across the finish line!

pnadolny13 avatar May 15 '24 20:05 pnadolny13

@pnadolny13 huge apologies! i had some health issues crop up and was out of office for a bit, and it's taking me some time to get fully recalibrated. i'm going to block out time Monday and Tuesday to catch up all the codegen things. Appreciate you!

gwenwindflower avatar May 30 '24 19:05 gwenwindflower

huge apologies! i had some health issues crop up and was out of office for a bit, and it's taking me some time to get fully recalibrated

No worries! Sorry to hear that, I hope everything is ok. Let me know if theres anything I can help with 😄

pnadolny13 avatar Jun 04 '24 14:06 pnadolny13

@pnadolny13 so i've tried a bunch of ways both before i went on medical leave and recently to get testing working for case sensitivity across all the warehouses and it just seems to be technically infeasible right now.

as such, i'm just removing testing and going to ship a beta release of case sensitivity without tests. if that seems to be okay after a month or so we'll just cut that as a regular release and keep going.

https://github.com/dbt-labs/dbt-adapters/issues/178 needs to be fixed in dbt Core to make testing possible for this, which is essentially we need for dbt to preserve case sensitivity in seeds, which it does not right now. at present if you have a CasEd SeEd dbt will default to the warehouse's default casing UPPER or lower, etc. So we're unable to seed the testing data reliably across adapters. C'est la vie, we shall beta it for now and it should be okay.

gwenwindflower avatar Jun 04 '24 16:06 gwenwindflower

@pnadolny13 please try out codegen 0.13.0-b1 and let me know it goes!

gwenwindflower avatar Jun 05 '24 15:06 gwenwindflower

linking https://github.com/dbt-labs/dbt-adapters/issues/178 as what screwed up the integration tests

dataders avatar Sep 10 '24 16:09 dataders