Fix Dashboard UI Hydrolix's connection problem (#103)

QuesmaOrg · May 14, 2024 · e77ab7d · e77ab7d
1 parent 64b700a
commit e77ab7d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/quesma/clickhouse/clickhouse.go b/quesma/clickhouse/clickhouse.go
@@ -477,6 +477,10 @@ func (lm *LogManager) AddTableIfDoesntExist(table *Table) bool {
 	return wasntCreated
 }
 
+func (lm *LogManager) Ping() error {
+	return lm.chDb.Ping()
+}
+
 func NewEmptyLogManager(cfg config.QuesmaConfiguration, chDb *sql.DB, phoneHomeAgent telemetry.PhoneHomeAgent) *LogManager {
 	ctx, cancel := context.WithCancel(context.Background())
 	var schemaManagement = NewSchemaManagement(chDb)

diff --git a/quesma/quesma/ui/dashboard_healthcheck.go b/quesma/quesma/ui/dashboard_healthcheck.go
@@ -1,7 +1,6 @@
 package ui
 
 import (
-	"github.com/ClickHouse/clickhouse-go/v2"
 	"net/http"
 	"sync"
 	"time"
@@ -55,9 +54,7 @@ func (qmc *QuesmaManagementConsole) checkClickhouseHealth() healthCheckStatus {
 	}
 
 	return qmc.clickhouseStatusCache.check(func() healthCheckStatus {
-		chDb := clickhouse.OpenDB(&clickhouse.Options{Addr: []string{qmc.config.ClickHouse.Url.Host}})
-		defer chDb.Close()
-		err := chDb.Ping()
+		err := qmc.logManager.Ping()
 		if err != nil {
 			return healthCheckStatus{"red", "Ping failed", err.Error()}