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? [ Full content available at: https://github.com/apache/incubator-superset/issues/5886 ] This message was relayed via gitbox.apache.org for [email protected]
