Format DeleteFiles JSON serde changes

manuzhang · codex · manuzhang · commit 00cb9fc2ed39 · 2026-05-21T20:25:41.000+08:00
Co-authored-by: Codex &lt;codex@openai.com&gt;
diff --git a/src/iceberg/json_serde.cc b/src/iceberg/json_serde.cc
@@ -1681,7 +1681,8 @@ Result<std::unique_ptr<TableUpdate>> TableUpdateFromJson(const nlohmann::json& j
     return std::make_unique<table::RemovePartitionStatistics>(snapshot_id);
   }
   if (action == kActionDeleteFiles) {
-    ICEBERG_ASSIGN_OR_RAISE(auto files, GetJsonValue<std::vector<std::string>>(json, kFiles));
+    ICEBERG_ASSIGN_OR_RAISE(auto files,
+                            GetJsonValue<std::vector<std::string>>(json, kFiles));
     return std::make_unique<table::DeleteFiles>(std::move(files));
   }
 
diff --git a/src/iceberg/test/json_serde_test.cc b/src/iceberg/test/json_serde_test.cc
@@ -650,9 +650,8 @@ TEST(JsonInternalTest, TableUpdateRemovePartitionStatistics) {
 }
 
 TEST(JsonInternalTest, TableUpdateDeleteFiles) {
-  table::DeleteFiles update(
-      {"s3://bucket/warehouse/table/metadata/old-1.json",
-       "s3://bucket/warehouse/table/metadata/old-2.json"});
+  table::DeleteFiles update({"s3://bucket/warehouse/table/metadata/old-1.json",
+                             "s3://bucket/warehouse/table/metadata/old-2.json"});
   nlohmann::json expected = R"({
     "action":"delete-files",
     "files":[

Original file line number	Diff line number	Diff line change
`@@ -1681,7 +1681,8 @@ Result<std::unique_ptr<TableUpdate>> TableUpdateFromJson(const nlohmann::json& j`
`1681`	`1681`	`return std::make_unique<table::RemovePartitionStatistics>(snapshot_id);`
`1682`	`1682`	`}`
`1683`	`1683`	`if (action == kActionDeleteFiles) {`
`1684`		`- ICEBERG_ASSIGN_OR_RAISE(auto files, GetJsonValue<std::vector<std::string>>(json, kFiles));`
	`1684`	`+ ICEBERG_ASSIGN_OR_RAISE(auto files,`
	`1685`	`+ GetJsonValue<std::vector<std::string>>(json, kFiles));`
`1685`	`1686`	`return std::make_unique<table::DeleteFiles>(std::move(files));`
`1686`	`1687`	`}`
`1687`	`1688`