transferwise / pipelinewise-target-redshift

Singer.io Target for Amazon Redshift - PipelineWise compatible
https://transferwise.github.io/pipelinewise/
Other
12 stars 65 forks source link

What is is with 'safe_column_name' forcing everything to upper case? #278

Open ian-lewis-d opened 1 year ago

ian-lewis-d commented 1 year ago

Forcing upper case for column names seems like an extremely narrow choice to make, especially where there is no option (it appears) to allow columns to be set to lower case.

We operate all our Redshift schemas with lower case entity names. Is there a way to avoid the forced upper case with target-redshift?

https://github.com/transferwise/pipelinewise-target-redshift/blob/80796b6c6e4647a85ddc9492e343547a027258e4/target_redshift/db_sync.py#L94