victornoel commented on issue #5886: Superset ignore case-sensitivity of time column names with Postgresql URL: https://github.com/apache/incubator-superset/issues/5886#issuecomment-423796775 @villebro I think there is a misunderstanding. The problem is with `col`, not with the period specifier. The fix should be quite simple: always put quotes around the column name when generating those query parts. This is, I believe, the standard SQL way of writing fields, see for example: https://www.jooq.org/doc/3.11/manual-single-page/#settings-name-style. Now the question is: are all db going to work with that? Maybe you can only fix this for postgresql (which use case sensitivity by default for column names, contrary to, say, sql server) for now, but normally this should be standard SQL.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
