use columns description instead of columns file. This should not break formats that support subset

arthurpassos · arthurpassos · commit 6ca505ef08ad · 2025-09-24T08:40:29.000-03:00
diff --git a/src/Storages/prepareReadingFromFormat.cpp b/src/Storages/prepareReadingFromFormat.cpp
@@ -101,7 +101,7 @@ ReadFromFormatInfo prepareReadingFromFormat(
     }
 
     /// Create header for InputFormat with columns that will be read from the data.
-    for (const auto & column : columns_in_data_file)
+    for (const auto & column : info.columns_description)
     {
         /// Never read hive partition columns from the data file. This fixes https://github.com/ClickHouse/ClickHouse/issues/87515
         if (!hive_parameters.hive_partition_columns_to_read_from_file_path_map.contains(column.name))

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ ReadFromFormatInfo prepareReadingFromFormat(`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`/// Create header for InputFormat with columns that will be read from the data.`
`104`		`- for (const auto & column : columns_in_data_file)`
	`104`	`+ for (const auto & column : info.columns_description)`
`105`	`105`	`{`
`106`	`106`	`/// Never read hive partition columns from the data file. This fixes https://github.com/ClickHouse/ClickHouse/issues/87515`
`107`	`107`	`if (!hive_parameters.hive_partition_columns_to_read_from_file_path_map.contains(column.name))`