deprecation of quoteIdentifiers
with Snowflake dialect
#16457
Replies: 1 comment
-
My knowledge of snowflake is minimal and we do not the possibility to run integration tests for it so all our support comes from the community. It seems we likely need to update the code mentioned below and document the migration path once the quoteIdentifiers option has been removed. If that will be a renamed snowflake only option or making use of the sequelize/packages/core/src/dialects/snowflake/query.js Lines 117 to 143 in 62f0419 Do note that v7 already has some major internal changes that have not been tested properly on snowflake so there's much more work to be done. See #15099 as well |
Beta Was this translation helpful? Give feedback.
-
Hi @WikiRik ,
regarding your comment here #2406 (comment)
I wonder how the deprecation of
quoteIdentifiers
will work together with Snowflake since the naming there is all uppercase.Without the
quoteIdentifiers: false
option - I need to rename all model fields to the uppercase form or use thefield
prop to map to the uppercased column.Is that right ? am I missing something ?
thank you!!
Beta Was this translation helpful? Give feedback.
All reactions