[AVRO-3970] [Rust] incorrect compatibility checks with logicalType uuid

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.12.0, 1.11.4
• Component/s: rust
• Labels: pull-request-available
• Assignee: Martin Tzvetanov Grigorov
• Reporter: mknaw
• Created: 04/Apr/24 23:22
• Updated: 10/Apr/24 08:25
• Resolved: 08/Apr/24 07:11

Description

A schema with a logicalType uuid field is incompatible with itself, per `SchemaCompatibility`:

 

use apache_avro::{Schema, schema_compatibility::SchemaCompatibility};

fn main() {     
    let schema_str = r#"\{"type": "string", "logicalType": "uuid"}"#;
    let writers_schema = Schema::parse_str(schema_str).unwrap();
    let readers_schema = Schema::parse_str(schema_str).unwrap();
    if let Err(err) = SchemaCompatibility::can_read(&writers_schema, &readers_schema) {
        eprintln!("Error: {}", err);
    }
}

 

this is the behavior on `main` as of writing (`876eae32`) and on the latest version on crates.rs (`v0.16.0`)

I have a proposed fix here: https://github.com/apache/avro/compare/main...mknaw:avro:logical-type-compat-fix

Perhaps there are other valid logicalTypes with this behavior. I have not checked every logicalType yet.

Issue Links

• links to: GitHub Pull Request #2838