AvroError enum for arrow-avro crate

[nathaniel-d-ef]

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

[jecsand838]

@nathaniel-d-ef I do like the idea of a dedicated error enum for arrow-avro. My only thought is whether we should handle like the non-parquet crates, i.e. in https://github.com/apache/arrow-rs/blob/main/arrow-schema/src/error.rs which does have an AvroError variant we can use.

My understanding is parquet is a bit different, since it's also the official Apache parquet implement for Rust. @alamb