add error context

apache · Dec 19, 2023 · 2f640bf · 2f640bf
1 parent e4438c1
commit 2f640bf
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/crates/iceberg/src/spec/manifest_list.rs b/crates/iceberg/src/spec/manifest_list.rs
@@ -694,7 +694,15 @@ pub(super) mod _serde {
                     .into_iter()
                     .map(|v| {
                         let partition_spec_id = v.partition_spec_id;
+                        let manifest_path = v.manifest_path.clone();
                         v.try_into(partition_types.get(&partition_spec_id))
+                            .map_err(|err| {
+                                err.with_context("manifest file path", manifest_path)
+                                    .with_context(
+                                        "partition spec id",
+                                        partition_spec_id.to_string(),
+                                    )
+                            })
                     })
                     .collect::<Result<Vec<_>, _>>()?,
             })
@@ -728,7 +736,15 @@ pub(super) mod _serde {
                     .into_iter()
                     .map(|v| {
                         let partition_spec_id = v.partition_spec_id;
+                        let manifest_path = v.manifest_path.clone();
                         v.try_into(partition_types.get(&partition_spec_id))
+                            .map_err(|err| {
+                                err.with_context("manifest file path", manifest_path)
+                                    .with_context(
+                                        "partition spec id",
+                                        partition_spec_id.to_string(),
+                                    )
+                            })
                     })
                     .collect::<Result<Vec<_>, _>>()?,
             })