anjakefala commented on code in PR #36266:
URL: https://github.com/apache/arrow/pull/36266#discussion_r1399691185


##########
python/pyarrow/tests/test_csv.py:
##########
@@ -1972,6 +1972,39 @@ def test_write_csv_decimal(tmpdir, type_factory):
     assert out.column('col').cast(type) == table.column('col')
 
 
[email protected]("data_size", (
+    int(1E2),
+    int(1E4),
+    int(1E6)

Review Comment:
   From playing with it, `int(1E6)` is the one that actually reproduces the bug 
(results in a segfault on main). 
   
   Is there value in also having `int(1E2)` or `int(1E4)` to make sure there 
wasn't a regression with relatively smaller binary arrays? If there isn't, we 
can remove those parameters. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to