(avro) branch main updated: AVRO-3779: [Rust] Fully specify the dependency to bigdecimal crate (#2595)

[mgrigorov]

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

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


The following commit(s) were added to refs/heads/main by this push:
     new c23c68bd4 AVRO-3779: [Rust] Fully specify the dependency to bigdecimal 
crate (#2595)
c23c68bd4 is described below

commit c23c68bd4bfa2f34c9053d93c2160fe3dd6f7f23
Author: Martin Grigorov <marti...@users.noreply.github.com>
AuthorDate: Wed Nov 22 09:19:18 2023 +0200

    AVRO-3779: [Rust] Fully specify the dependency to bigdecimal crate (#2595)
    
    AVRO-3779: Simplify the schema to test only BigDecimal
    
    Depend on full version and enable only the `std` feature
    
    Signed-off-by: Martin Tzvetanov Grigorov <mgrigo...@apache.org>
---
 lang/rust/Cargo.lock         |  4 ++--
 lang/rust/avro/Cargo.toml    |  2 +-
 lang/rust/avro/src/schema.rs | 11 +++++------
 3 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/lang/rust/Cargo.lock b/lang/rust/Cargo.lock
index d585c8fb4..b50a6383c 100644
--- a/lang/rust/Cargo.lock
+++ b/lang/rust/Cargo.lock
@@ -158,9 +158,9 @@ dependencies = [
 
 [[package]]
 name = "bigdecimal"
-version = "0.4.1"
+version = "0.4.2"
 source = "registry+https://github.com/rust-lang/crates.io-index";
-checksum = "454bca3db10617b88b566f205ed190aedb0e0e6dd4cad61d3988a72e8c5594cb"
+checksum = "c06619be423ea5bb86c95f087d5707942791a08a85530df0db2209a3ecfb8bc9"
 dependencies = [
  "autocfg",
  "libm",
diff --git a/lang/rust/avro/Cargo.toml b/lang/rust/avro/Cargo.toml
index b9610a8b5..60cf9da41 100644
--- a/lang/rust/avro/Cargo.toml
+++ b/lang/rust/avro/Cargo.toml
@@ -55,6 +55,7 @@ name = "single"
 
 [dependencies]
 apache-avro-derive = { default-features = false, version = "0.17.0", path = 
"../avro_derive", optional = true }
+bigdecimal = { default-features = false, version = "0.4.2", features = ["std"] 
}
 bzip2 = { default-features = false, version = "0.4.4", optional = true }
 crc32fast = { default-features = false, version = "1.3.2", optional = true }
 digest = { default-features = false, version = "0.10.7", features = 
["core-api"] }
@@ -73,7 +74,6 @@ typed-builder = { default-features = false, version = 
"0.16.2" }
 uuid = { default-features = false, version = "1.4.1", features = ["serde", 
"std"] }
 xz2 = { default-features = false, version = "0.1.7", optional = true }
 zstd = { default-features = false, version = "0.12.4+zstd.1.5.2", optional = 
true }
-bigdecimal = "0.4"
 
 [target.'cfg(target_arch = "wasm32")'.dependencies]
 quad-rand = { default-features = false, version = "0.2.1" }
diff --git a/lang/rust/avro/src/schema.rs b/lang/rust/avro/src/schema.rs
index bc371b5e0..f2487e316 100644
--- a/lang/rust/avro/src/schema.rs
+++ b/lang/rust/avro/src/schema.rs
@@ -5155,17 +5155,12 @@ mod tests {
     #[test]
     fn test_avro_3779_bigdecimal_schema() -> TestResult {
         let schema = json!(
-        {
-          "type": "record",
-          "name": "recordWithDecimal",
-          "fields": [
             {
                 "name": "decimal",
                 "type": "bytes",
                 "logicalType": "big-decimal"
             }
-          ]
-        });
+        );
 
         let parse_result = Schema::parse(&schema);
         assert!(
@@ -5173,6 +5168,10 @@ mod tests {
             "parse result must be ok, got: {:?}",
             parse_result
         );
+        match parse_result? {
+            Schema::BigDecimal => (),
+            other => panic!("Expected Schema::BigDecimal but got: {other:?}"),
+        }
 
         Ok(())
     }