Remove pointless sleep from legacy workfile_mgr_test
It reduces the test runtime by 1 minute.  It must have been a remnant
of an old refactor.  I couldn't find a justification for it in last
few years worth of commit history.
@@ -704,13 +704,6 @@ workfile_fill_sharedcache(void)
success = false;
break;
}
- if (crt_entry >= gp_workfile_max_entries - 2)
- {
- /* Pause between adding extra ones so we can test from other sessions */
- elog(LOG, "Added %d entries out of %d, pausing for 30 seconds before proceeding", crt_entry + 1, n_entries);
- sleep(30);
- }
-
}
unit_test_result(success);
