@mahendra Sorry for not having been able to extend my final comments, The time window that I had reserved for reviewing this was wasted rebuilding my dev env, and this week ended up being busier than I had expected, hence the added delay. I will do my best to give my final comments asap.
With regards to merging, this is also a feature my organisation is eagerly awaiting. However, I am not officially affiliated with the project, and don't have merge rights. So consider my comments as reflecting only the needs of my organisation, not the original Superset dev team. [ Full content available at: https://github.com/apache/incubator-superset/pull/5294 ] This message was relayed via gitbox.apache.org for [email protected]
