Apply suggestions from code review - Joris

Co-authored-by: Joris Van den Bossche <[email protected]>
apache · Jun 20, 2023 · 08297d0 · 08297d0
1 parent 9a2631b
commit 08297d0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/python/pyarrow/src/arrow/python/inference.cc b/python/pyarrow/src/arrow/python/inference.cc
@@ -79,7 +79,7 @@ Status ImportPresentIntervalTypes(OwnedRefNoGIL* interval_types_tuple) {
 
 }  // namespace
 
-int import_pyarrow();
+import_pyarrow();
 
 #define _NUMPY_UNIFY_NOOP(DTYPE) \
   case NPY_##DTYPE:              \

diff --git a/python/pyarrow/tests/test_convert_builtin.py b/python/pyarrow/tests/test_convert_builtin.py
@@ -2380,8 +2380,8 @@ def test_array_from_pylist_offset_overflow():
     ([pa.MonthDayNano([1, -1, -10100])], [pa.scalar(pa.MonthDayNano([1, -1, -10100]))]),
     (["a", "b"], [pa.scalar("a"), pa.scalar("b")]),
     ([b"a", b"b"], [pa.scalar(b"a"), pa.scalar(b"b")]),
-    ([1, 2, 3], pa.scalar([1, 2, 3])),
-    (["a", "b"], pa.scalar(["a", "b"])),
+    ([[1, 2, 3]], [pa.scalar([1, 2, 3])]),
+    ([["a", "b"]], [pa.scalar(["a", "b"])]),
 ])
 def test_array_accepts_pyarrow_scalar(seq, data, scalar_data):
     if type(seq(scalar_data)) == set: