GH-44918: [Ruby] Fix a bug that empty list is ignored in builder

apache · Dec 4, 2024 · 70abedb · 70abedb
1 parent 50f2d6e
commit 70abedb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/ruby/red-arrow/lib/arrow/list-array-builder.rb b/ruby/red-arrow/lib/arrow/list-array-builder.rb
@@ -54,8 +54,8 @@ def append_value(*args)
         when nil
           append_null
         when ::Array
-          return if value.empty?
           append_value_raw
+          return if value.empty?
           @value_builder ||= value_builder
           @value_builder.append(*value)
         else

diff --git a/ruby/red-arrow/test/test-list-array-builder.rb b/ruby/red-arrow/test/test-list-array-builder.rb
@@ -40,7 +40,7 @@ def setup
       builder = Arrow::ListArrayBuilder.new(data_type)
       builder.append_value([])
       array = builder.finish
-      assert_equal([], array[0].to_a)
+      assert_equal([[]], array.to_a)
     end
   end