Merge pull request #50369 from ClickHouse/backport/23.4/50232

Backport #50232 to 23.4: Fix iceberg metadata parsing
ClickHouse · May 31, 2023 · 3ce5413 · 3ce5413
2 parents edcd8ed + 7519fdb
commit 3ce5413
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/Storages/DataLakes/IcebergMetadataParser.cpp b/src/Storages/DataLakes/IcebergMetadataParser.cpp
@@ -243,7 +243,13 @@ struct IcebergMetadataParser<Configuration, MetadataReadHelper>::Impl
 
             const auto * str_col = assert_cast<const ColumnString *>(col_str.get());
             for (size_t i = 0; i < str_col->size(); ++i)
-                keys.emplace_back(str_col->getDataAt(i).toView());
+            {
+                const auto data_path = std::string(str_col->getDataAt(i).toView());
+                const auto pos = data_path.find(configuration.url.key);
+                if (pos == std::string::npos)
+                    throw Exception(ErrorCodes::BAD_ARGUMENTS, "Expected to find {} in data path: {}", configuration.url.key, data_path);
+                keys.emplace_back(data_path.substr(pos));
+            }
         }
 
         return keys;