This is an automated email from the ASF dual-hosted git repository.
zeroshade pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg-cpp.git
The following commit(s) were added to refs/heads/main by this push:
new 25d4d62 fix: content in manifest entry should be optional (#172)
25d4d62 is described below
commit 25d4d626af4bf5527289acd5c6e56524da09c00a
Author: nullccxsy <[email protected]>
AuthorDate: Thu Aug 14 08:56:26 2025 +0800
fix: content in manifest entry should be optional (#172)
kContent in manifest entry should be optional, not required
Co-authored-by: nullccxsy <[email protected]>
---
src/iceberg/manifest_entry.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/iceberg/manifest_entry.h b/src/iceberg/manifest_entry.h
index 36fd8f8..0aa697a 100644
--- a/src/iceberg/manifest_entry.h
+++ b/src/iceberg/manifest_entry.h
@@ -176,7 +176,7 @@ struct ICEBERG_EXPORT DataFile {
/// present
std::optional<int64_t> content_size_in_bytes;
- inline static const SchemaField kContent = SchemaField::MakeRequired(
+ inline static const SchemaField kContent = SchemaField::MakeOptional(
134, "content", iceberg::int32(),
"Contents of the file: 0=data, 1=position deletes, 2=equality deletes");
inline static const SchemaField kFilePath = SchemaField::MakeRequired(
