OpenRefine / OpenRefine / 18886627020

51%

push

github

fix loading 100 rows in db import preview (#7507) Fixes #7499

* cover loading multiple batches in preview and import

* rm comments with log statements

* divide code into commented sections

* fix off-by-one error

* make sure dbRows is not null

* encapsulate handling of end in getRowsOfCells

* keep ignoring end flag in db import for now

* add log statements for debugging

* sync changes in preview and import

* rm additional preloading of batches

* simplify progress tracking

* replace preview reader with import reader

* replace with ref to empty list

* refactor construction of header into one-liner

* switch to lazy eval of logging parameter

* rename boolean end for clarity

* make parameters final

* fix parameter in error log

* rm IOException from method signature as it is never thrown

* switch to 0-based logging for batches

5694 of 12746 branches covered (44.67%)

Branch coverage included in aggregate %.

14481 of 27033 relevant lines covered (53.57%)

2.49 hits per line