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]

Reply via email to