Speed improvement - Avoid unnecessary copy of a part of an ArrayList

forcedotcom · Nov 26, 2024 · 4ccafa9 · 4ccafa9
1 parent ddb1a67
commit 4ccafa9
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/src/main/java/com/salesforce/dataloader/action/visitor/bulk/BulkLoadVisitor.java b/src/main/java/com/salesforce/dataloader/action/visitor/bulk/BulkLoadVisitor.java
@@ -492,10 +492,7 @@ private void processResults(final DataReader dataReader, final BatchInfo batch,
         final int totalRowsInDAOInCurrentBatch = lastDAORowForCurrentBatch - this.firstDAORowForCurrentBatch + 1;
         List<TableRow> rows;
         if (controller.getAppConfig().getBoolean(AppConfig.PROP_PROCESS_BULK_CACHE_DATA_FROM_DAO)) {
-            rows = new ArrayList<TableRow>();
-            for (int i=0; i<totalRowsInDAOInCurrentBatch; i++) {
-                rows.add(i, this.daoRowList.get(i + this.firstDAORowForCurrentBatch));
-            }
+            rows = this.daoRowList.subList(firstDAORowForCurrentBatch, firstDAORowForCurrentBatch+totalRowsInDAOInCurrentBatch);
         } else {
             rows = dataReader.readTableRowList(totalRowsInDAOInCurrentBatch);
         }