Fokko opened a new issue, #1587:
URL: https://github.com/apache/iceberg-rust/issues/1587
### Is your feature request related to a problem or challenge?
For reading Manifest/ManifestList using PyIceberg we want to have the
interface as simple as possible. Therefore we want to enable reading V1
metadata (manifest-list/manifest) using a V2 reader. This is what we do today
in PyIceberg, and it makes upgrading a table to a newer version much easier.
When trying to read all the manifests though Rust, I'm seeing the following
error:
```
@cached(cache=LRUCache(maxsize=128), key=lambda io, manifest_list:
hashkey(manifest_list))
def _manifests(io: FileIO, manifest_list: str) -> Tuple[ManifestFile,
...]:
"""Read and cache manifests from the given manifest list, returning
a tuple to prevent modification."""
bs = io.new_input(manifest_list).open().read()
from pyiceberg_core import manifest
> entries = list(manifest.read_manifest_list(bs).entries())
E pyo3_runtime.PanicException: called `Result::unwrap()` on an `Err`
value: DataInvalid => Failure in conversion with avro
E
E Source: Failed to deserialize Avro value into value: missing field
`content`
```
This one refers to `517: content`:
<img width="712" height="1459" alt="Image"
src="https://github.com/user-attachments/assets/934405ad-46b7-4dcc-8e4f-14271ae5425b"
/>
The comment sais: use 0 for all v1 manifests. Therefore we can set the
default value of Avro to 0 to populate the field in the case of V1
manifest-lists.
### Describe the solution you'd like
Able to read a V1 manifest-list using a V2 reader.
### Willingness to contribute
None
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]