fix: allow PartitionField's field_id to be missing in Iceberg v1 (#121)

Just looks like forget to use `allow_field_id_missing` in
`PartitionFieldFromJson(const nlohmann::json& json, bool
allow_field_id_missing)`

In v1 iceberg, we should allow field_id is missing.

---------

Signed-off-by: Smith Cruise <[email protected]>

Author: Smith-Cruise

src/iceberg/json_internal.cc

@@ -1056,7 +1056,9 @@ Status ParsePartitionSpecs(const nlohmann::json& json, int8_t format_version,
     int32_t next_partition_field_id = PartitionSpec::kLegacyPartitionDataIdStart;
     std::vector<PartitionField> fields;
     for (const auto& entry_json : partition_spec_json) {
-      ICEBERG_ASSIGN_OR_RAISE(auto field, PartitionFieldFromJson(entry_json));
+      ICEBERG_ASSIGN_OR_RAISE(
+          auto field, PartitionFieldFromJson(
+                          entry_json, /*allow_field_id_missing=*/format_version == 1));
       int32_t field_id = field->field_id();
       if (field_id == SchemaField::kInvalidFieldId) {
         // If the field ID is not set, we need to assign a new one