[issue-5397] [BE] address review: use Dropwizard Duration for healthCheckTimeout

- Change healthCheckTimeout field type from int to Dropwizard Duration in
  DatabaseAnalyticsFactory, allowing flexible config values (e.g. 1s, 500ms)
- Rename @nAmed binding to snake_case "health_check_timeout"
- Update config.yml and config-test.yml to use Duration string format
- Env var renamed from ANALYTICS_DB_HEALTH_CHECK_TIMEOUT_SECONDS to
  ANALYTICS_DB_HEALTH_CHECK_TIMEOUT

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: dsblank

apps/opik-backend/config.yml

@@ -86,9 +86,9 @@ databaseAnalytics:
   #   - custom_http_params=max_query_size=100000000
   #  Description: query parameters that will be added to the connection string
   queryParameters: ${ANALYTICS_DB_QUERY_PARAMETERS:-health_check_interval=2000&compress=1&auto_discovery=true&failover=3&custom_http_params=max_query_size=100000000,async_insert_busy_timeout_max_ms=250,async_insert_busy_timeout_min_ms=100,async_insert=1,wait_for_async_insert=1,async_insert_use_adaptive_busy_timeout=1,async_insert_deduplicate=1,use_skip_indexes_if_final=1}
-  #  Default: 1
-  #  Description: Timeout in seconds for the ClickHouse health check query
-  healthCheckTimeoutSeconds: ${ANALYTICS_DB_HEALTH_CHECK_TIMEOUT_SECONDS:-1}
+  #  Default: 1s
+  #  Description: Timeout for the ClickHouse health check query (e.g. 1s, 500ms, 2 minutes)
+  healthCheckTimeout: ${ANALYTICS_DB_HEALTH_CHECK_TIMEOUT:-1s}
 
 # https://www.dropwizard.io/en/stable/manual/configuration.html#health
 health:

apps/opik-backend/src/main/java/com/comet/opik/infrastructure/DatabaseAnalyticsFactory.java

@@ -1,5 +1,6 @@
 package com.comet.opik.infrastructure;
 
+import io.dropwizard.util.Duration;
 import io.r2dbc.spi.ConnectionFactories;
 import io.r2dbc.spi.ConnectionFactory;
 import jakarta.validation.constraints.NotBlank;
@@ -20,7 +21,7 @@ public class DatabaseAnalyticsFactory {
     private @NotNull String password;
     private @NotBlank String databaseName;
     private String queryParameters;
-    private int healthCheckTimeoutSeconds = 1;
+    private Duration healthCheckTimeout = Duration.seconds(1);
 
     public ConnectionFactory build() {
         var options = queryParameters == null ? "" : "?%s".formatted(queryParameters);

apps/opik-backend/src/main/java/com/comet/opik/infrastructure/db/ClickHouseHealthyCheck.java

@@ -1,25 +1,24 @@
 package com.comet.opik.infrastructure.db;
 
+import io.dropwizard.util.Duration;
 import jakarta.inject.Inject;
 import jakarta.inject.Named;
 import jakarta.inject.Singleton;
 import lombok.NonNull;
 import reactor.core.publisher.Mono;
 import ru.vyarus.dropwizard.guice.module.installer.feature.health.NamedHealthCheck;
 
-import java.time.Duration;
-
 @Singleton
 public class ClickHouseHealthyCheck extends NamedHealthCheck {
 
     private final TransactionTemplateAsync template;
-    private final Duration healthCheckTimeout;
+    private final java.time.Duration healthCheckTimeout;
 
     @Inject
     public ClickHouseHealthyCheck(@NonNull TransactionTemplateAsync template,
-            @Named("ClickHouse Health Check Timeout Seconds") int healthCheckTimeoutSeconds) {
+            @Named("health_check_timeout") Duration healthCheckTimeout) {
         this.template = template;
-        this.healthCheckTimeout = Duration.ofSeconds(healthCheckTimeoutSeconds);
+        this.healthCheckTimeout = java.time.Duration.ofMillis(healthCheckTimeout.toMilliseconds());
     }
 
     @Override

apps/opik-backend/src/main/java/com/comet/opik/infrastructure/db/DatabaseAnalyticsModule.java

@@ -5,6 +5,7 @@
 import com.comet.opik.infrastructure.OpikConfiguration;
 import com.comet.opik.infrastructure.log.UserFacingLoggingFactory;
 import com.google.inject.Provides;
+import io.dropwizard.util.Duration;
 import io.opentelemetry.api.GlobalOpenTelemetry;
 import io.opentelemetry.instrumentation.r2dbc.v1_0.R2dbcTelemetry;
 import io.r2dbc.spi.ConnectionFactory;
@@ -46,9 +47,9 @@ public String getDatabaseName() {
 
     @Provides
     @Singleton
-    @Named("ClickHouse Health Check Timeout Seconds")
-    public int getHealthCheckTimeoutSeconds() {
-        return databaseAnalyticsFactory.getHealthCheckTimeoutSeconds();
+    @Named("health_check_timeout")
+    public Duration getHealthCheckTimeout() {
+        return databaseAnalyticsFactory.getHealthCheckTimeout();
     }
 
     @Provides

apps/opik-backend/src/test/resources/config-test.yml

@@ -70,9 +70,9 @@ databaseAnalytics:
   #   - custom_http_params=max_query_size=100000000
   #  Description: query parameters that will be added to the connection string
   queryParameters: health_check_interval=2000&compress=1&auto_discovery=true&failover=3&custom_http_params=max_query_size=100000000,async_insert_busy_timeout_max_ms=60,async_insert_busy_timeout_min_ms=50,async_insert=1,wait_for_async_insert=1,async_insert_use_adaptive_busy_timeout=1,async_insert_deduplicate=1,use_skip_indexes_if_final=1
-  #  Default: 1
-  #  Description: Timeout in seconds for the ClickHouse health check query
-  healthCheckTimeoutSeconds: 1
+  #  Default: 1s
+  #  Description: Timeout for the ClickHouse health check query (e.g. 1s, 500ms, 2 minutes)
+  healthCheckTimeout: 1s
 
 # https://www.dropwizard.io/en/stable/manual/configuration.html#health
 health: