You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@avro.apache.org by mg...@apache.org on 2022/11/18 09:59:42 UTC

[avro] 01/01: AVRO-3674: Fix the build for Rust 1.54 (MSRV)

This is an automated email from the ASF dual-hosted git repository.

mgrigorov pushed a commit to branch avro-3674
in repository https://gitbox.apache.org/repos/asf/avro.git

commit fbe9a76a1b4ece70702ef2a208e3abe97949c21c
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
AuthorDate: Fri Nov 18 11:54:50 2022 +0200

    AVRO-3674: Fix the build for Rust 1.54 (MSRV)
    
    Add support for all named schema types
    
    Signed-off-by: Martin Tzvetanov Grigorov <mg...@apache.org>
---
 lang/rust/avro/src/types.rs | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/lang/rust/avro/src/types.rs b/lang/rust/avro/src/types.rs
index c328f1d68..fadff4481 100644
--- a/lang/rust/avro/src/types.rs
+++ b/lang/rust/avro/src/types.rs
@@ -341,14 +341,9 @@ impl Value {
     pub fn validate(&self, schema: &Schema) -> bool {
         let rs = ResolvedSchema::try_from(schema).expect("Schema didn't successfully parse");
         let namespace = match schema {
-            Schema::Record {
-                name,
-                aliases: _,
-                doc: _,
-                fields: _,
-                lookup: _,
-                attributes: _,
-            } => &name.namespace,
+            Schema::Record { name, .. }
+            | Schema::Enum { name, .. }
+            | Schema::Fixed { name, .. } => &name.namespace,
             _ => &None,
         };
 
@@ -589,10 +584,10 @@ impl Value {
         match *schema {
             Schema::Ref { ref name } => {
                 if let Some(resolved) = names.get(name) {
-                    info!("Resolved {name:?}");
+                    info!("Resolved {:?}", name);
                     self.resolve_internal(resolved, names)
                 } else {
-                    info!("Failed to resolve schema {name:?}");
+                    error!("Failed to resolve schema {:?}", name);
                     Err(Error::SchemaResolutionError(name.clone()))
                 }
             }