Code review comment: Question, should we have this changed for everything using mini cluster, or only for our tests that are using unaligned checkpoints? 🤔   Maybe for example because of benchmarks, and overall running times (costs of our CI), we should keep using memory storage if not needed? 

Code before review:
private void startMiniCluster() throws Exception {
final Configuration configuration =
new Configuration(miniClusterResourceConfiguration.getConfiguration());
configuration.setString(
CoreOptions.TMP_DIRS, temporaryFolder.newFolder().getAbsolutePath());
configuration.set(
CheckpointingOptions.CHECKPOINTS_DIRECTORY,
temporaryFolder.newFolder().toURI().toString());
if (!configuration.contains(CoreOptions.FILESYTEM_DEFAULT_OVERRIDE)) {
configuration.setBoolean(CoreOptions.FILESYTEM_DEFAULT_OVERRIDE, true);
}
if (!configuration.contains(TaskManagerOptions.MANAGED_MEMORY_SIZE)) {
configuration.set(TaskManagerOptions.MANAGED_MEMORY_SIZE, DEFAULT_MANAGED_MEMORY_SIZE);
}
configuration.setInteger(JobManagerOptions.PORT, 0);
if (!(configuration.contains(RestOptions.BIND_PORT)
|| configuration.contains(RestOptions.PORT))) {
configuration.setString(RestOptions.BIND_PORT, "0");
}
randomizeConfiguration(configuration);
final MiniClusterConfiguration miniClusterConfiguration =
new MiniClusterConfiguration.Builder()
.setConfiguration(configuration)
.setNumTaskManagers(
miniClusterResourceConfiguration.getNumberTaskManagers())
.setNumSlotsPerTaskManager(
miniClusterResourceConfiguration.getNumberSlotsPerTaskManager())
.setRpcServiceSharing(
miniClusterResourceConfiguration.getRpcServiceSharing())
.setHaServices(miniClusterResourceConfiguration.getHaServices())
.build();
miniCluster =
new MiniCluster(miniClusterConfiguration, () -> Reference.borrowed(rpcSystem));
miniCluster.start();
final URI restAddress = miniCluster.getRestAddress().get();
createClientConfiguration(restAddress);
}

Line under review: temporaryFolder.newFolder().toURI().toString());

Code after addressing the review:
private void startMiniCluster() throws Exception {
final Configuration configuration =
new Configuration(miniClusterResourceConfiguration.getConfiguration());
configuration.setString(
CoreOptions.TMP_DIRS, temporaryFolder.newFolder().getAbsolutePath());
if (!configuration.contains(CheckpointingOptions.CHECKPOINTS_DIRECTORY)) {
configuration.set(
CheckpointingOptions.CHECKPOINTS_DIRECTORY,
temporaryFolder.newFolder().toURI().toString());
}
if (!configuration.contains(CoreOptions.FILESYTEM_DEFAULT_OVERRIDE)) {
configuration.setBoolean(CoreOptions.FILESYTEM_DEFAULT_OVERRIDE, true);
}
if (!configuration.contains(TaskManagerOptions.MANAGED_MEMORY_SIZE)) {
configuration.set(TaskManagerOptions.MANAGED_MEMORY_SIZE, DEFAULT_MANAGED_MEMORY_SIZE);
}
configuration.setInteger(JobManagerOptions.PORT, 0);
if (!(configuration.contains(RestOptions.BIND_PORT)
|| configuration.contains(RestOptions.PORT))) {
configuration.setString(RestOptions.BIND_PORT, "0");
}
randomizeConfiguration(configuration);
final MiniClusterConfiguration miniClusterConfiguration =
new MiniClusterConfiguration.Builder()
.setConfiguration(configuration)
.setNumTaskManagers(
miniClusterResourceConfiguration.getNumberTaskManagers())
.setNumSlotsPerTaskManager(
miniClusterResourceConfiguration.getNumberSlotsPerTaskManager())
.setRpcServiceSharing(
miniClusterResourceConfiguration.getRpcServiceSharing())
.setHaServices(miniClusterResourceConfiguration.getHaServices())
.build();
miniCluster =
new MiniCluster(miniClusterConfiguration, () -> Reference.borrowed(rpcSystem));
miniCluster.start();
final URI restAddress = miniCluster.getRestAddress().get();
createClientConfiguration(restAddress);
}