Closed tbembersimeao closed 6 years ago
FWIW - Haven't looked at the code, but the rationale for 'fields per batch' instead of 'records per batch' was to handle cases in testing that had thousands of fields per record where batching by record in a reasonable number could easily overwhelm the memory limits...
@123andy that makes sense, I confess I didn't get it at first glance. I just committed "fields per batch" option back. Thank you!
Improvements:
export.php
,progress_bar.php
(template),download.php
,css/batch.css
,js/batch.js
Review steps:
sleep(5);
in/plugins/ajax/batch.php
, then reduce the number of fields per batch, and then run export process againsleep()
when calculating timeouts)