Skip to content

Commit

Permalink
refactor!: rename store redis connectionTimeout to timeout
Browse files Browse the repository at this point in the history
  • Loading branch information
mattnichols committed Mar 17, 2023
1 parent 26a0506 commit a204f47
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ final synchronized StatefulRedisConnection<String, String> buildConnection() {
.computationThreadPoolSize(configuration.getComputationThreadPoolSize())
.build();

RedisClient redisClient = RedisClient.create(resources, new RedisURI(configuration.getHost(), configuration.getPort(), configuration.getConnectionTimeout()));
RedisClient redisClient = RedisClient.create(resources, new RedisURI(configuration.getHost(), configuration.getPort(), configuration.getTimeout()));

// RESP3 executes a HELLO command to discover the protocol before executing any commands made by the client.
// This can cause issues if we are communicating over a proxy and the proxy doesn't speak RESP3. For now, it is safer
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ public class RedisStoreConfiguration {

private static final String DEFAULT_HOST = "localhost";
private static final int DEFAULT_COMPUTATION_THREAD_POOL_SIZE = 5;
private static final Duration DEFAULT_CONNECTION_TIMEOUT = Duration.ofSeconds(10);
private static final Duration DEFAULT_TIMEOUT = Duration.ofSeconds(10);
private static final int DEFAULT_PORT = 6379;
private static final int DEFAULT_THREAD_POOL_SIZE = 5;

Expand All @@ -22,7 +22,7 @@ public class RedisStoreConfiguration {
private int computationThreadPoolSize = DEFAULT_COMPUTATION_THREAD_POOL_SIZE;

@ConfigurationField
private Duration connectionTimeout = DEFAULT_CONNECTION_TIMEOUT;
private Duration timeout = DEFAULT_TIMEOUT;

@ConfigurationField
private int ioThreadPoolSize = DEFAULT_THREAD_POOL_SIZE;
Expand Down
1 change: 1 addition & 0 deletions upgrades/v3.0.0/facilities-3.0.0-upgrade.sh
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ function process_file {
"com.mx.path.service.facility.fault_tolerant_executor([^a-zA-Z])=com.mx.path.service.facility.fault_tolerant_executor.resilience4j\1"
"com.mx.path.facility.exception_reporter.honeybadger([^a-zA-Z])=com.mx.path.service.facility.exception_reporter.honeybadger\1"
"com.mx.path.facility.security.vault([^a-zA-Z])=com.mx.path.service.facility.security.vault\1"
"connectionTimeout([^a-zA-Z])=timeout\1"
)

local reported_file=false
Expand Down

0 comments on commit a204f47

Please sign in to comment.