[arrow] branch master updated: ARROW-8808: [Rust] Fix divide by zero error in builder

[nevime]

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

nevime pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new 7194f94  ARROW-8808: [Rust] Fix divide by zero error in builder
7194f94 is described below

commit 7194f94e0ed65d9429180eb96e35fcb4eca38848
Author: Andy Grove <andygrov...@gmail.com>
AuthorDate: Sat May 16 09:51:23 2020 +0200

    ARROW-8808: [Rust] Fix divide by zero error in builder
    
    Fix divide by zero error in one of the builders. This bug was exposed by 
the integration tests.
    
    Closes #7186 from andygrove/ARROW-8808
    
    Lead-authored-by: Andy Grove <andygrov...@gmail.com>
    Co-authored-by: Wakahisa <nevilled...@gmail.com>
    Signed-off-by: Neville Dipale <nevilled...@gmail.com>
---
 rust/arrow/src/array/builder.rs | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/rust/arrow/src/array/builder.rs b/rust/arrow/src/array/builder.rs
index b689e2d..9952fdd 100644
--- a/rust/arrow/src/array/builder.rs
+++ b/rust/arrow/src/array/builder.rs
@@ -1862,6 +1862,16 @@ mod tests {
     }
 
     #[test]
+    fn test_fixed_size_list_array_builder_empty() {
+        let values_builder = Int32Array::builder(5);
+        let mut builder = FixedSizeListBuilder::new(values_builder, 3);
+
+        let arr = builder.finish();
+        assert_eq!(0, arr.len());
+        assert_eq!(0, builder.len());
+    }
+
+    #[test]
     fn test_fixed_size_list_array_builder_finish() {
         let values_builder = Int32Array::builder(5);
         let mut builder = FixedSizeListBuilder::new(values_builder, 3);