Michael Weinberg

this is an example of a general suggestion (calling this a suggestion, not an issue, since really nothing is broken) https://github.com/chanzuckerberg/terraform-provider-snowflake/blob/3d29ed6dbca4f2c6d147fd260300c0bada96fedb/pkg/snowflake/future_grant.go#L40 https://docs.snowflake.com/en/sql-reference/identifier-literal.html This is an opinion, but forced use of...

Support group_labels in yml for dimensions

1

https://github.com/lightdash/dbt2looker/blob/bb8f5b485ec541e2b1be15363ac3c7f8f19d030d/dbt2looker/models.py#L99 measures seem to have this but not dimensions. probably all/most properties in available in https://docs.lightdash.com/references/dimensions/ should be represented here -- is this something lightdash is willing to maintain or...

min_block_sec and max_block_sec aggregate different measures

1

https://github.com/awslabs/amazon-redshift-utils/blob/bfe6b021e3c292723f52e4c7f81c5278e37737c4/src/AdminViews/v_get_blocking_locks.sql#L64 Is this really correct? Why use transaction time for one and waiting time for the other? annotations on this view would be great, because I don't understand it well...

[Feature] dynamic tables: support macros in "snowflake_warehouse" config value

4

### Is this your first time submitting a feature request? - [X] I have read the [expectations for open source contributors](https://docs.getdbt.com/docs/contributing/oss-expectations) - [X] I have searched the existing issues, and...

https://github.com/dbt-labs/dbt-external-tables/blob/4d26c38c81ac7c39e9ed8f50792b43de769e88cb/macros/plugins/snowflake/snowpipe/get_copy_sql.sql#L32 it seems to me that this line could be optional, bc stages can store file formats and the file_format parameter is optional in COPY INTO (per reference docs)