Took a quick look. I think the original author wrote this because that if no `schema` is provided on the frontend, the backend will receive an empty string which is `''`, but it should be `None` otherwise the database will throw some kind of error like `schema does not exist`. So yes, I think it is just casting falsy values, specifically `''` to `None`.
[ Full content available at: https://github.com/apache/incubator-superset/pull/5787 ] This message was relayed via gitbox.apache.org for [email protected]
