refactor: wrap UNION query in subquery for correct ORDER BY, LIMIT, and OFFSET application; update test documentation

Author: lohanidamodar

src/Usage/Adapter/ClickHouse.php

@@ -1009,17 +1009,21 @@ public function find(array $queries = []): array
         $offsetClause = isset($parsed['offset']) ? ' OFFSET {offset:UInt64}' : '';
 
         // Query both tables with UNION ALL
+        // Wrap in subquery to ensure ORDER BY, LIMIT, OFFSET apply to the entire UNION result
         $sql = "
-            SELECT {$selectColumns}
-            FROM {$fromTable}{$whereClause}
-            UNION ALL
-            SELECT {$selectColumns}
-            FROM {$fromCounterTable}{$whereClause}
-            {$orderClause}{$limitClause}{$offsetClause}
+            SELECT *
+            FROM (
+                SELECT {$selectColumns}
+                FROM {$fromTable}{$whereClause}
+                UNION ALL
+                SELECT {$selectColumns}
+                FROM {$fromCounterTable}{$whereClause}
+            ){$orderClause}{$limitClause}{$offsetClause}
             FORMAT JSON
         ";
 
         $result = $this->query($sql, $parsed['params']);
+
         return $this->parseResults($result);
     }
 
@@ -1352,6 +1356,9 @@ private function parseResults(string $result): array
                 if ($key === 'tenant') {
                     // Parse tenant
                     $document[$key] = $value !== null ? (int) $value : null;
+                } elseif ($key === 'value') {
+                    // Parse value as integer
+                    $document[$key] = $value !== null ? (int) $value : null;
                 } elseif ($key === 'time') {
                     // Time comes as string in JSON format, convert to ISO 8601 if needed
                     $parsedTime = (string)$value;

tests/Usage/Adapter/ClickHouseTest.php

@@ -79,8 +79,8 @@ public function testMetricTenantOverridesAdapterTenantInBatch(): void
     }
 
     /**
-    * Test logBatch with explicit batch size parameter
-    */
+     * Test logBatch with explicit batch size parameter
+     */
     public function testLogBatchWithBatchSize(): void
     {
         $metrics = [