Nj-kol commented on pull request #10084: URL: https://github.com/apache/incubator-superset/pull/10084#issuecomment-654004803
> @Nj-kol this fix would be important to get merged before cutting the 0.37 release. If you feel uneasy adding the unit test, please feel free to remove the test so we can get this merged. I'll be happy to reintroduce the test later, as we need to address similar tests for other engines, too. Yes. I have kind of been struggling with getting the tests to pass, largely because of my lack of experience in Python. If this is urgent and know can do without the tests, I'll remove them ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
