mistercrunch commented on PR #28268: URL: https://github.com/apache/superset/pull/28268#issuecomment-2968396286
Oh wow, this has been open for a moment, seems this whole method needs some serious refactoring, breaking out something like `handle_resultset_special_types` or something of that nature. @betodealmeida may have some input/guidance here, but this method is calls for a refactor as it's at the limit of what a single method should try to handle... -- 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
