Skip to content

Commit

Permalink
ADAPT-1433 Adjust cache mechanism within applications (#873)
Browse files Browse the repository at this point in the history
*minor fix for test in dp-composer

Co-authored-by: Mykola Nikulesko <[email protected]>
  • Loading branch information
mnikulesko and Mykola Nikulesko authored Sep 15, 2023
1 parent 5123d0c commit 74ae346
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ class RedisSchemaRegistryClient(restService: RestService,
httpHeaders: Map[String, String],
configs: Map[String, Any],
cacheConfigs: CacheConfigs,
ssl: Boolean = true) extends SchemaRegistryClient {
ssl: Boolean) extends SchemaRegistryClient {

def this(baseUrl: String, redisHost: String, redisPort: Int) {
this(new RestService(baseUrl), redisHost, redisPort, Map.empty[String, String], Map.empty[String, Any], CacheConfigs(1, 1, 1), true)
Expand Down
2 changes: 2 additions & 0 deletions core/src/main/resources/reference.conf
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@ hydra {
schema.registry.redis.use-redis = ${?HYDRA_SCHEMA_REGISTRY_USE_REDIS}
schema.registry.redis.host = ${?HYDRA_SCHEMA_REGISTRY_REDIS_HOST}
schema.registry.redis.port = ${?HYDRA_SCHEMA_REGISTRY_REDIS_PORT}
schema.registry.redis.ssl = true
schema.registry.redis.ssl = ${?HYDRA_SCHEMA_REGISTRY_REDIS_SSL}
schema.registry.redis.id-cache-ttl = 1
schema.registry.redis.id-cache-ttl = ${?HYDRA_SCHEMA_REGISTRY_REDIS_ID_CACHE_TTL}
schema.registry.redis.schema-cache-ttl = 1
Expand Down
2 changes: 1 addition & 1 deletion ingest/src/main/scala/hydra.ingest/app/AppConfig.scala
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ object AppConfig {
env("HYDRA_SCHEMA_REGISTRY_REDIS_PORT")
.as[Int]
.default(6379),
env("HYDRA_SCHEMA_REGISTRY_USE_SSL")
env("HYDRA_SCHEMA_REGISTRY_REDIS_SSL")
.as[Boolean]
.default(true),
env("HYDRA_SCHEMA_REGISTRY_REDIS_ID_CACHE_TTL")
Expand Down

0 comments on commit 74ae346

Please sign in to comment.