search

electric-sql / legacy-cli

ElectricSQL command line interface (CLI) tool.
https://electric-sql.com/docs
Apache License 2.0
10 stars 0 forks source link

[VAX-430] enforcing only lowercase names for columns #13

Closed paulharter closed 1 year ago

define-null commented 1 year ago

Ticket is missing

balegas commented 1 year ago

Can someone (@paulharter) please take action to close this? Open ticket, request review and/or hand-off to someone if necessary

balegas commented 1 year ago

Ticket for this is VAX-430.

magnetised commented 1 year ago

can you have a look @paulharter, thanks

linear[bot] commented 1 year ago
VAX-430 Postgres DDL case-sensitive

I am using some old migrations I have generated and I haven't put column names inside quotes. What happens is that column names are stored all lower case in Postgres and this causes column names not to be found when Electric tries to store rows. * I don't know if this has been handled meanwhile * I think we can store case-sensitive names with single quotes, or we just convert names appropriately * Please ensure Electric works accordingly with the decision here