Vitor-Avila commented on PR #35832: URL: https://github.com/apache/superset/pull/35832#issuecomment-3561369304
thanks @gabotorresruiz, left a few more comments with the latest changes 🙏 one important detail is that **currently we support caching the list of channels without celery workers**. This PR on the other hand limits caching to only instances that have Celery workers configured -- @eschutho do you think this a concern? I would imagine all large organizations that would really need several calls to the Slack APIs would have celery configured, but wondering if we want to keep this functionality. -- 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]
