From 53e660c13855651d73ae6268c6b6f55b3d37cb42 Mon Sep 17 00:00:00 2001 From: Yahor Berdnikau Date: Wed, 12 Oct 2022 18:00:44 +0200 Subject: [PATCH] Fix benchmark aggregation on different measured builds number When scenarios have different iterations number, resulted csv file will contain empty entries for some of the measurements. Aggregation should ignore such entries. --- .../org/jetbrains/kotlin/gradle/benchmark/BenchmarkTemplate.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libraries/tools/gradle/regression-benchmark-templates/src/main/kotlin/org/jetbrains/kotlin/gradle/benchmark/BenchmarkTemplate.kt b/libraries/tools/gradle/regression-benchmark-templates/src/main/kotlin/org/jetbrains/kotlin/gradle/benchmark/BenchmarkTemplate.kt index 1f218eff3cf..db8d1211560 100644 --- a/libraries/tools/gradle/regression-benchmark-templates/src/main/kotlin/org/jetbrains/kotlin/gradle/benchmark/BenchmarkTemplate.kt +++ b/libraries/tools/gradle/regression-benchmark-templates/src/main/kotlin/org/jetbrains/kotlin/gradle/benchmark/BenchmarkTemplate.kt @@ -376,7 +376,7 @@ abstract class BenchmarkTemplate( ) columnName.startsWith("measured build") -> DataColumn.createValueColumn( columnName, - rowToColumn(columnName) { it.toString().toInt() } + rowToColumn(columnName) { it.toString().toIntOrNull() } ) else -> throw IllegalArgumentException("Unknown column name: $columnName") }