michaelneely
michaelneely
I'm getting the same sequence of errors. What I tried: 1. add ` sql_cmds[:-1]` to line 144 2. change `[]` to `set()` on lines 166, 167 3. change `ProjectProto.FromString(row[1]["project_proto"])` to...
There are a lot of configuration options (see [here](https://github.com/snowflakedb/snowflake-connector-python/blob/07230cff1e8cb86f63bf52a0c4c3a14b083ba448/src/snowflake/connector/connection.py#L172)). I think trying to maintain parity for dozens of options would be a difficult endeavour. Users could easily refer to the...
I think the parameters already explicitly listed in the Snowflake models are considered the "standard" options. Allowing extra parameters increases the flexibility for advanced use cases. Maybe we could mention...
Another option would be to add an explicit "connection extras" (not sure on name) parameter that we unpack in the `GetSnowflakeConnection` class.
Under the proposed approach of allowing extras, if I wanted to add the `host` parameter to my snowflake connection, then my `feature_store.yml` would look like this: ```yml registry: registry_type: snowflake.registry...
Gotcha. I think it would be exactly like above, except `host` would sit under `connection_extras`, e.g.,: ```yaml registry: registry_type: snowflake.registry account: ... user: ... password: ... role: ... warehouse: ......
Thanks, @jyejare, how can I get this merged?
@pintaoz-aws - Could you take a look at this PR? Thanks!