Fix accidental revert of import batch size
authorEileen McNaughton <emcnaughton@wikimedia.org>
Thu, 20 Oct 2022 02:50:54 +0000 (15:50 +1300)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Thu, 20 Oct 2022 02:50:54 +0000 (15:50 +1300)
commitcdd62b15369be4b6736d1822500824d8d26c902f
tree6f8c08a22a2c79e086c85a938121c6235d1b1547
parent1e956573c5d22e3223be88ce0532e6a824640a20
Fix accidental revert of import batch size

Overview
----------------------------------------
We increased batch size to 50 in 5.52 but appear to have accidentally
re-set it to 5. This was presumably due to code being merged that
patched the code with the older value & hadn't picked up the new

Before
----------------------------------------
Each batch is 5 records

After
----------------------------------------
Each batch is 50 records

Technical Details
----------------------------------------
There seems to be consensus this should be at least 50.
100 is also on the table but I went with Andy's comment
https://github.com/civicrm/civicrm-core/pull/24151#issuecomment-1284498891

The issue Andy points out about the 3 second delay between batches seems excessive
and worth digging into. There used to be a 30 second delay in the UI just to
let people really luxuriate in watching our batch screen - not sure
if that is still there?
CRM/Import/Parser.php