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-425620932 FYI, the PR was closed, because, I quote @rumbin: >I am concerned that this PR will again revive #3844. From the issues and PRs within this very issue you can see that this topic is very controversial. > >That time we came to these conclusions: > > - Quoting the time column name breaks calculated time columns. > - A workaround for Case-sensitive time columns may be to rename them by creating a calculated column out of them. I would be great to tackle this situation properly by stopping generating SQL queries by hand and use the proper tools (i.e. SQLAlchemy) even if it's "less readable". WDYT @villebro @rumbin @mistercrunch?
---------------------------------------------------------------- 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]
