cemremengu opened a new pull request, #23157: URL: https://github.com/apache/superset/pull/23157
### SUMMARY Fixes #21827 When a database is created `schema_cache_timeout` property is set to empty string by default inside `extra_json` (or at least this is the case for older versions) which leads to the following error when calling `get_all_schema_names` which exptects an optional integer. This PR fixes this error. ``` redis.exceptions.ResponseError: value is not an integer or out of range ``` Additionally, these params don't seem to be used in `get_all_schema_names` anymore. So maybe the best course is to remove them all together. ### TESTING INSTRUCTIONS Without this, the SQL Lab shows an error when fetching schemas -- 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: notifications-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
