giftig commented on PR #23852: URL: https://github.com/apache/superset/pull/23852#issuecomment-1532587160
Fixed the pylint errors; looks like I missed an import when porting the fix from our fork to the base project, and I've added pylint exceptions for local imports. I didn't make the original change but it looks like the `DatabaseError` was local in the presto class because it had been done locally in another place there, so I'm assuming this was necessary. The mysql test error looks like a flaky test, don't think it's anything to do with my change. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
