villebro commented on issue #14619: URL: https://github.com/apache/superset/issues/14619#issuecomment-1922262673
I may also be hazy on the details, but IIRC, `Sentinel` is a wrapper that maintains an instance to a `Redis` instance, which then gets swapped out during failover. For this reason, I think the correct solution would be to make the class configurable, so that GAQ could use any caching backend. Having said that, I seem to recall that the GAQ implementation uses some Redis specific logic, so it may well be that we can't use any arbitrary caching backend, like Memcached. But I would need to check this. -- 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
