@rumbin good point, I didn't know the `col` in this expression could have been more complex things.
And indeed a workaround is to exploit that situation to make a calculated column with a proper name. Isn't the actual real solution to the problem to stop trying to manipulate SQL by hand and use the proper tools so that this situation does not arise? According to @villebro, using sqlalchemy to generate those queries would be "not too readable" (cf https://github.com/apache/incubator-superset/issues/5886#issuecomment-423795666) but at least it would be correct in all cases, right? I will close this for now, but I would really appreciate if you, @rumbin, could chip in the discussion in #5886 so that we find a solution to this situation? [ Full content available at: https://github.com/apache/incubator-superset/pull/5962 ] This message was relayed via gitbox.apache.org for [email protected]
