fix: org-scoped ClickHouse issues

- Remove unused schema fields (keepAliveEnabled, keepAliveIdleSocketTtl, maxOpenConnections)
- Move ClickHouse lookup inside deferred error handling in logs route

These fixes address review comments from PR triggerdotdev#3333.

Author: deepshekhardas

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.logs/route.tsx

@@ -133,26 +133,27 @@ export const loader = async ({ request, params }: LoaderFunctionArgs) => {
   const from = fromStr ? parseInt(fromStr, 10) : undefined;
   const to = toStr ? parseInt(toStr, 10) : undefined;
 
-  // Get the user's plan to determine log retention limit
+// Get the user's plan to determine log retention limit
   const plan = await getCurrentPlan(project.organizationId);
   const retentionLimitDays = plan?.v3Subscription?.plan?.limits.logRetentionDays.number ?? 30;
 
-  const logsClickhouse = await clickhouseFactory.getClickhouseForOrganization(project.organizationId, "logs");
-  const presenter = new LogsListPresenter($replica, logsClickhouse);
-
-  const listPromise = presenter
-    .call(project.organizationId, environment.id, {
-      userId,
-      projectId: project.id,
-      tasks: tasks.length > 0 ? tasks : undefined,
-      runId,
-      search,
-      levels,
-      period,
-      from,
-      to,
-      defaultPeriod: "1h",
-      retentionLimitDays,
+  const listPromise = clickhouseFactory
+    .getClickhouseForOrganization(project.organizationId, "logs")
+    .then((logsClickhouse) => {
+      const presenter = new LogsListPresenter($replica, logsClickhouse);
+      return presenter.call(project.organizationId, environment.id, {
+        userId,
+        projectId: project.id,
+        tasks: tasks.length > 0 ? tasks : undefined,
+        runId,
+        search,
+        levels,
+        period,
+        from,
+        to,
+        defaultPeriod: "1h",
+        retentionLimitDays,
+      });
     })
     .catch((error) => {
       if (error instanceof ServiceValidationError) {

apps/webapp/app/routes/admin.api.v1.runs-replication.create.ts

@@ -11,9 +11,6 @@ import {
 
 const CreateRunReplicationServiceParams = z.object({
   name: z.string(),
-  keepAliveEnabled: z.boolean(),
-  keepAliveIdleSocketTtl: z.number(),
-  maxOpenConnections: z.number(),
   maxFlushConcurrency: z.number(),
   flushIntervalMs: z.number(),
   flushBatchSize: z.number(),