[GitHub] [incubator-superset] villebro commented on pull request #9859: fix: raise error in sqllab when using reserved column name

[GitBox <gi...@apache.org>]

villebro commented on pull request #9859:
URL: https://github.com/apache/incubator-superset/pull/9859#issuecomment-632517957


   @eugeniamz the thing here is that the query as such is not really usable for re-exploration, as it has time grains, filters etc burned into the query. My interpretation is that this feature is used for throwaway analysis purposes only. Perhaps we should also add a comment above the query to that effect to explain that the query is only a reflection of a certain chart, and should usually not be used as a basis for a new chart?


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org