fix: MV column not under aggregate - use subquery alias for toStartOfDay

Author: lohanidamodar

src/Usage/Adapter/ClickHouse.php

@@ -973,20 +973,21 @@ public function setup(): void
         $escapedDailyMv = $this->escapeIdentifier($this->database) . '.' . $this->escapeIdentifier($dailyMvName);
 
         $tenantSelect = $this->sharedTables ? 'tenant' : "'' as tenant";
-        $groupByClause = $this->sharedTables ? 'metric, tenant' : 'metric';
+        $groupByClause = $this->sharedTables ? 'metric, tenant, day' : 'metric, day';
 
         $dailySelect = $this->sharedTables
-            ? "generateUUIDv4() as id, metric, {$tenantSelect}, sum(value) as value, 'event' as type, toStartOfDay(time) as time, '{}' as tags"
-            : "generateUUIDv4() as id, metric, sum(value) as value, 'event' as type, toStartOfDay(time) as time, '{}' as tags";
+            ? "generateUUIDv4() as id, metric, {$tenantSelect}, sum(value) as value, 'event' as type, day as time, '{}' as tags"
+            : "generateUUIDv4() as id, metric, sum(value) as value, 'event' as type, day as time, '{}' as tags";
 
         $createDailyMvSql = "
             CREATE MATERIALIZED VIEW IF NOT EXISTS {$escapedDailyMv}
             TO {$escapedDailyTable}
             AS SELECT
+                toStartOfDay(time) as day,
                 {$dailySelect}
             FROM {$escapedDatabaseAndTable}
             WHERE type = 'event'
-            GROUP BY {$groupByClause}, toStartOfDay(time)
+            GROUP BY {$groupByClause}
         ";
 
         $this->query($createDailyMvSql);