cyb70289 commented on a change in pull request #11598:
URL: https://github.com/apache/arrow/pull/11598#discussion_r742472350
##########
File path: python/pyarrow/tests/test_csv.py
##########
@@ -1321,8 +1321,17 @@ def test_cancellation(self):
raise_signal = util.get_raise_signal()
# Make the interruptible workload large enough to not finish
- # before the interrupt comes, even in release mode on fast machines
- large_csv = b"a,b,c\n" + b"1,2,3\n" * 200_000_000
+ # before the interrupt comes, even in release mode on fast machines.
+ last_duration = 0.0
+ workload_size = 100_000
+
+ while last_duration < 1.0:
+ print("workload size:", workload_size)
Review comment:
Looks pytest hides the output by default. Curious to see the workload
size big enough for M1.
##########
File path: python/pyarrow/tests/test_csv.py
##########
@@ -1321,8 +1321,17 @@ def test_cancellation(self):
raise_signal = util.get_raise_signal()
# Make the interruptible workload large enough to not finish
- # before the interrupt comes, even in release mode on fast machines
- large_csv = b"a,b,c\n" + b"1,2,3\n" * 200_000_000
+ # before the interrupt comes, even in release mode on fast machines.
+ last_duration = 0.0
+ workload_size = 100_000
+
+ while last_duration < 1.0:
+ print("workload size:", workload_size)
Review comment:
Looks pytest hides the output by default. Curious to see the workload
size big enough for M1 in CI pool.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
