raise the default Redis timeout to 500ms

.changesets/config_geal_raise_redis_timeouts.md

@@ -0,0 +1,5 @@
+### Increase default Redis timeout ([PR #5795](https://github.com/apollographql/router/pull/5795))
+
+The default Redis command timeout was increased from 2ms to 500ms to accommodate common production use cases.
+
+By [@Geal](https://github.com/Geal) in https://github.com/apollographql/router/pull/5795

Cargo.toml

@@ -75,4 +75,4 @@ serde_json_bytes = { version = "0.2.4", features = ["preserve_order"] }
 sha1 = "0.10.6"
 tempfile = "3.10.1"
 tokio = { version = "1.36.0", features = ["full"] }
-tower = { version = "0.4.13", features = ["full"] }
+tower = { version = "0.4.13", features = ["full"] }

apollo-router/src/cache/redis.rs

@@ -171,7 +171,7 @@ impl RedisCacheStorage {
         let client = RedisClient::new(
             client_config,
             Some(PerformanceConfig {
-                default_command_timeout: config.timeout.unwrap_or(Duration::from_millis(2)),
+                default_command_timeout: config.timeout.unwrap_or(Duration::from_millis(500)),
                 ..Default::default()
             }),
             None,

docs/source/configuration/distributed-caching.mdx

@@ -131,7 +131,7 @@ supergraph:
         urls: ["redis://..."] #highlight-line
         username: admin/123 # Optional, can be part of the urls directly, mainly useful if you have special character like '/' in your password that doesn't work in url. This field takes precedence over the username in the URL
         password: admin # Optional, can be part of the urls directly, mainly useful if you have special character like '/' in your password that doesn't work in url. This field takes precedence over the password in the URL
-        timeout: 5ms # Optional, by default: 2ms
+        timeout: 2s # Optional, by default: 500ms
         ttl: 24h # Optional
         namespace: "prefix"  # Optional
         #tls:
@@ -141,7 +141,7 @@ supergraph:
 
 #### Timeout
 
-Connecting and sending commands to Redis are subject to a timeout, set by default to 2ms, that can be overriden.
+Connecting and sending commands to Redis are subject to a timeout, set by default to 500ms, that can be overriden.
 
 
 #### TTL

docs/source/configuration/entity-caching.mdx

@@ -91,7 +91,7 @@ preview_entity_cache:
       # Configure Redis
       redis:
         urls: ["redis://..."]
-        timeout: 5ms # Optional, by default: 2ms
+        timeout: 2s # Optional, by default: 500ms
         ttl: 24h # Optional, by default no expiration
     # Configure entity caching per subgraph, overrides options from the "all" section
     subgraphs: