villebro edited a comment on issue #6986: BUG: Time series Line chart has broken URL: https://github.com/apache/incubator-superset/issues/6986#issuecomment-477691089 Did changing the parameter work? I think there are two options: 1. Add a note to the documentation that `QUOTED_IDENTIFIERS_IGNORE_CASE` needs to be set to False. This is what I believe the majority of users are doing anyway. 2. Change the spec for Snowflake to always uppercase column labels. I'm kind of leaning towards the second option, as it makes Snowflake "just work" out of the box in both cases, has no adverse effects (other than making the labels in queries look VERY_LOUD), and makes it easier to get Superset working with minimal additional configuration. Thoughts?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to 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]
