nathaniel-d-ef opened a new issue, #8746:
URL: https://github.com/apache/arrow-rs/issues/8746

   **Is your feature request related to a problem or challenge? Please describe 
what you are trying to do.**
   
   Newly published crate arrow-avro is missing an AvroError enum, which it 
should have to follow the pattern of Parquet and others in the project. This 
became evident when attempting to integrate the new crate into DataFusion, 
which imports Error from apache-arrow as AvroError, for which there is no clean 
replacement here.
   
   **Describe the solution you'd like**
   
   A new AvroError enum should be created following the same pattern as 
ParquetError. Then:
   
   - Functions inside the arrow-avro crate should be modified to return 
Result<..., AvroError>. This includes functions for parsing, compression, and 
schema validation.
   - Functions that are part of the crate's public API (e.g., Reader::next(), 
Writer::write()) should be modified to return Result<..., ArrowError>.
   
   **Describe alternatives you've considered**
   
   Nothing's broken with the current implementation, it's just not as specific 
as it could be, and doesn't fully follow the patterns of the project.
   
   cc @jecsand838 
   


-- 
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]

Reply via email to