Merge pull request ClickHouse#63453 from ClickHouse/fix-coverage

alexey-milovidov · web-flow · commit 37fe5da9e426 · 2024-05-07T13:46:28.000Z
Fix collection of coverage data in the CI Logs cluster
diff --git a/docker/test/base/setup_export_logs.sh b/docker/test/base/setup_export_logs.sh
@@ -140,7 +140,7 @@ function setup_logs_replication
             time DateTime COMMENT 'The time of test run',
             test_name String COMMENT 'The name of the test',
             coverage Array(UInt64) COMMENT 'An array of addresses of the code (a subset of addresses instrumented for coverage) that were encountered during the test run'
-        ) ENGINE = Null COMMENT 'Contains information about per-test coverage from the CI, but used only for exporting to the CI cluster'
+        ) ENGINE = MergeTree ORDER BY test_name COMMENT 'Contains information about per-test coverage from the CI, but used only for exporting to the CI cluster'
     "
 
     # For each system log table:

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,7 @@ function setup_logs_replication`
`140`	`140`	`time DateTime COMMENT 'The time of test run',`
`141`	`141`	`test_name String COMMENT 'The name of the test',`
`142`	`142`	`coverage Array(UInt64) COMMENT 'An array of addresses of the code (a subset of addresses instrumented for coverage) that were encountered during the test run'`
`143`		`- ) ENGINE = Null COMMENT 'Contains information about per-test coverage from the CI, but used only for exporting to the CI cluster'`
	`143`	`+ ) ENGINE = MergeTree ORDER BY test_name COMMENT 'Contains information about per-test coverage from the CI, but used only for exporting to the CI cluster'`
`144`	`144`	`"`
`145`	`145`
`146`	`146`	`# For each system log table:`