FrankYang0529 commented on code in PR #71067: URL: https://github.com/apache/airflow/pull/71067#discussion_r3796779218
########## providers/redis/docs/connections.rst: ########## @@ -31,22 +32,30 @@ parameter as ``redis_default`` by default. Configuring the Connection -------------------------- Host - The host of the Redis cluster. + The host of the Redis server. Port - Specify the port to use for connecting the Redis cluster (Default is ``6379``). + Specify the port to use for connecting the Redis server (Default is ``6379``). Login - The user that will be used for authentication against the Redis cluster (only applicable in Redis 6.0 and above). + The user that will be used for authentication against the Redis server (only applicable in Redis 6.0 and above). Password - The password of the user that will be used for authentication against the Redis cluster. + The password of the user that will be used for authentication against the Redis server. DB - The DB number to use in the Redis cluster (Default is ``0``). + The DB number to use in the Redis server (Default is ``0``). Not supported in cluster mode. + +Enable cluster mode + Whether to connect with a cluster-aware client that follows ``MOVED``/``ASK`` redirects + (Default is ``False``). See :ref:`redis-cluster-mode` below. Review Comment: Thanks for the great suggestion. Yes, the `Is cluster` is more clear. -- 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]
